You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by td...@apache.org on 2018/01/19 18:03:44 UTC
[36/50] [abbrv] phoenix git commit: PHOENIX-4460 High GC / RS
shutdown when we use select query with IN clause using 4.10 phoenix client on
4.13 phoenix server
PHOENIX-4460 High GC / RS shutdown when we use select query with IN clause using 4.10 phoenix client on 4.13 phoenix server
Project: http://git-wip-us.apache.org/repos/asf/phoenix/repo
Commit: http://git-wip-us.apache.org/repos/asf/phoenix/commit/9d8be0e9
Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/9d8be0e9
Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/9d8be0e9
Branch: refs/heads/system-catalog
Commit: 9d8be0e9214ba3680a81c399c5da316c1b91c99b
Parents: 5cb02da
Author: Thomas D'Silva <td...@apache.org>
Authored: Thu Dec 14 15:41:42 2017 -0800
Committer: Thomas D'Silva <td...@apache.org>
Committed: Mon Dec 18 20:28:15 2017 -0800
----------------------------------------------------------------------
.../apache/phoenix/coprocessor/BaseScannerRegionObserver.java | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/phoenix/blob/9d8be0e9/phoenix-core/src/main/java/org/apache/phoenix/coprocessor/BaseScannerRegionObserver.java
----------------------------------------------------------------------
diff --git a/phoenix-core/src/main/java/org/apache/phoenix/coprocessor/BaseScannerRegionObserver.java b/phoenix-core/src/main/java/org/apache/phoenix/coprocessor/BaseScannerRegionObserver.java
index 519e419..316a286 100644
--- a/phoenix-core/src/main/java/org/apache/phoenix/coprocessor/BaseScannerRegionObserver.java
+++ b/phoenix-core/src/main/java/org/apache/phoenix/coprocessor/BaseScannerRegionObserver.java
@@ -366,7 +366,10 @@ abstract public class BaseScannerRegionObserver extends BaseRegionObserver {
if (scan.isRaw() || ScanInfoUtil.isKeepDeletedCells(store.getScanInfo()) || scan.getTimeRange().getMax() == HConstants.LATEST_TIMESTAMP || TransactionUtil.isTransactionalTimestamp(scan.getTimeRange().getMax())) {
return s;
}
-
+
+ if (s!=null) {
+ s.close();
+ }
ScanInfo scanInfo = ScanInfoUtil.cloneScanInfoWithKeepDeletedCells(store.getScanInfo());
return new StoreScanner(store, scanInfo, scan, targetCols,
c.getEnvironment().getRegion().getReadpoint(scan.getIsolationLevel()));