phoenix-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jamestay...@apache.org
Subject git commit: PHOENIX-1130 SkipScanFilter gets IndexOutOfBoundsException when intersecting salted tables (Kyle Buzsaki)
Date Wed, 30 Jul 2014 00:39:56 GMT
Repository: phoenix
Updated Branches:
  refs/heads/4.0 b61d182f4 -> f7f470528


PHOENIX-1130 SkipScanFilter gets IndexOutOfBoundsException when intersecting salted tables
(Kyle Buzsaki)


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

Branch: refs/heads/4.0
Commit: f7f47052866a186ce316b317dbbdccb03170b0ac
Parents: b61d182
Author: James Taylor <jamestaylor@apache.org>
Authored: Tue Jul 29 17:15:37 2014 -0700
Committer: James Taylor <jamestaylor@apache.org>
Committed: Tue Jul 29 17:38:58 2014 -0700

----------------------------------------------------------------------
 .../src/main/java/org/apache/phoenix/filter/SkipScanFilter.java    | 2 ++
 1 file changed, 2 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/phoenix/blob/f7f47052/phoenix-core/src/main/java/org/apache/phoenix/filter/SkipScanFilter.java
----------------------------------------------------------------------
diff --git a/phoenix-core/src/main/java/org/apache/phoenix/filter/SkipScanFilter.java b/phoenix-core/src/main/java/org/apache/phoenix/filter/SkipScanFilter.java
index a6b8161..5d23376 100644
--- a/phoenix-core/src/main/java/org/apache/phoenix/filter/SkipScanFilter.java
+++ b/phoenix-core/src/main/java/org/apache/phoenix/filter/SkipScanFilter.java
@@ -219,6 +219,8 @@ public class SkipScanFilter extends FilterBase implements Writable {
             if (Arrays.equals(lowerPosition, position) && areSlotsSingleKey(0, position.length-1))
{
                 return false;
             }
+        } else if (filterAllRemaining()) {
+            return true;
         }
         // Copy inclusive all positions 
         for (int i = 0; i <= lastSlot; i++) {


Mime
View raw message