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 2016/07/28 20:57:05 UTC

phoenix git commit: PHOENIX-3121 Mention dependency on HBASE-16296

Repository: phoenix
Updated Branches:
  refs/heads/4.x-HBase-0.98 2384f6b2f -> 3fc406698


PHOENIX-3121 Mention dependency on HBASE-16296


Project: http://git-wip-us.apache.org/repos/asf/phoenix/repo
Commit: http://git-wip-us.apache.org/repos/asf/phoenix/commit/3fc40669
Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/3fc40669
Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/3fc40669

Branch: refs/heads/4.x-HBase-0.98
Commit: 3fc406698d90c7b5a8a12342a3d080bf902e1bb9
Parents: 2384f6b
Author: Samarth <sa...@salesforce.com>
Authored: Thu Jul 28 13:54:40 2016 -0700
Committer: Samarth <sa...@salesforce.com>
Committed: Thu Jul 28 13:54:40 2016 -0700

----------------------------------------------------------------------
 .../src/main/java/org/apache/phoenix/execute/BaseQueryPlan.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/phoenix/blob/3fc40669/phoenix-core/src/main/java/org/apache/phoenix/execute/BaseQueryPlan.java
----------------------------------------------------------------------
diff --git a/phoenix-core/src/main/java/org/apache/phoenix/execute/BaseQueryPlan.java b/phoenix-core/src/main/java/org/apache/phoenix/execute/BaseQueryPlan.java
index f6c7be4..ff44d2e 100644
--- a/phoenix-core/src/main/java/org/apache/phoenix/execute/BaseQueryPlan.java
+++ b/phoenix-core/src/main/java/org/apache/phoenix/execute/BaseQueryPlan.java
@@ -236,8 +236,8 @@ public abstract class BaseQueryPlan implements QueryPlan {
         
         if (OrderBy.REV_ROW_KEY_ORDER_BY.equals(orderBy)) {
             ScanUtil.setReversed(scan);
-            // Hack for working around PHOENIX-3121 (reverse scans fail when limit is a multiple of scanner cache size).
-            //TODO: remove once PHOENIX-3121 is fixed.
+            // Hack for working around PHOENIX-3121 and HBASE-16296.
+            // TODO: remove once PHOENIX-3121 and/or HBASE-16296 are fixed.
             int scannerCacheSize = context.getStatement().getFetchSize();
             if (limit != null && limit % scannerCacheSize == 0) {
                 scan.setCaching(scannerCacheSize + 1);