You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by sa...@apache.org on 2017/11/03 00:52:38 UTC
phoenix git commit: PHOENIX-4287 Add null check for parent name
Repository: phoenix
Updated Branches:
refs/heads/master 7d2205d0c -> 895d06797
PHOENIX-4287 Add null check for parent name
Project: http://git-wip-us.apache.org/repos/asf/phoenix/repo
Commit: http://git-wip-us.apache.org/repos/asf/phoenix/commit/895d0679
Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/895d0679
Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/895d0679
Branch: refs/heads/master
Commit: 895d067974639cd2205b14940e4e46864b4e2060
Parents: 7d2205d
Author: Samarth Jain <sa...@apache.org>
Authored: Thu Nov 2 17:52:32 2017 -0700
Committer: Samarth Jain <sa...@apache.org>
Committed: Thu Nov 2 17:52:32 2017 -0700
----------------------------------------------------------------------
.../java/org/apache/phoenix/iterate/BaseResultIterators.java | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/phoenix/blob/895d0679/phoenix-core/src/main/java/org/apache/phoenix/iterate/BaseResultIterators.java
----------------------------------------------------------------------
diff --git a/phoenix-core/src/main/java/org/apache/phoenix/iterate/BaseResultIterators.java b/phoenix-core/src/main/java/org/apache/phoenix/iterate/BaseResultIterators.java
index 18f28e2..eb09813 100644
--- a/phoenix-core/src/main/java/org/apache/phoenix/iterate/BaseResultIterators.java
+++ b/phoenix-core/src/main/java/org/apache/phoenix/iterate/BaseResultIterators.java
@@ -1246,9 +1246,10 @@ public abstract class BaseResultIterators extends ExplainTable implements Result
}
/*
* For a view index, we use the property set on view. For indexes on base table, whether
- * global or local, we use the property set on the base table.
+ * global or local, we use the property set on the base table. Null check needed when
+ * dropping local indexes.
*/
- if (table.getType() == PTableType.INDEX) {
+ if (table.getType() == PTableType.INDEX && table.getParentName() != null) {
PhoenixConnection conn = context.getConnection();
String parentTableName = table.getParentName().getString();
try {