You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by el...@apache.org on 2018/03/23 02:21:58 UTC
[2/6] phoenix git commit: PHOENIX-4662 Avoid NPE when server-caches
are null (Csaba Skrabak)
PHOENIX-4662 Avoid NPE when server-caches are null (Csaba Skrabak)
Project: http://git-wip-us.apache.org/repos/asf/phoenix/repo
Commit: http://git-wip-us.apache.org/repos/asf/phoenix/commit/d1c48b6d
Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/d1c48b6d
Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/d1c48b6d
Branch: refs/heads/4.x-HBase-1.3
Commit: d1c48b6d7a903f3a7a87222bcf1cf514b062f83c
Parents: 45e75de
Author: Josh Elser <el...@apache.org>
Authored: Thu Mar 22 16:03:21 2018 -0400
Committer: Josh Elser <el...@apache.org>
Committed: Thu Mar 22 16:31:19 2018 -0400
----------------------------------------------------------------------
.../java/org/apache/phoenix/iterate/TableResultIterator.java | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/phoenix/blob/d1c48b6d/phoenix-core/src/main/java/org/apache/phoenix/iterate/TableResultIterator.java
----------------------------------------------------------------------
diff --git a/phoenix-core/src/main/java/org/apache/phoenix/iterate/TableResultIterator.java b/phoenix-core/src/main/java/org/apache/phoenix/iterate/TableResultIterator.java
index e812854..f6902cc 100644
--- a/phoenix-core/src/main/java/org/apache/phoenix/iterate/TableResultIterator.java
+++ b/phoenix-core/src/main/java/org/apache/phoenix/iterate/TableResultIterator.java
@@ -198,9 +198,12 @@ public class TableResultIterator implements ResultIterator {
retry--;
try {
Long cacheId = ((HashJoinCacheNotFoundException) e1).getCacheId();
+
+ ServerCache cache = caches == null ? null :
+ caches.get(new ImmutableBytesPtr(Bytes.toBytes(cacheId)));
+
if (!hashCacheClient.addHashCacheToServer(newScan.getStartRow(),
- caches.get(new ImmutableBytesPtr(Bytes.toBytes(cacheId))),
- plan.getTableRef().getTable())) {
+ cache, plan.getTableRef().getTable())) {
throw e1;
}
this.scanIterator = ((BaseQueryPlan) plan).iterator(caches, scanGrouper, newScan);