PHOENIX-6323 isRawFilterSupported() returns true for HBase 2.1 comapt…

… module
apache · Jan 18, 2021 · cdee24d · cdee24d
1 parent 50df995
commit cdee24d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/...e-compat-2.1.6/src/main/java/org/apache/phoenix/compat/hbase/HbaseCompatCapabilities.java b/...e-compat-2.1.6/src/main/java/org/apache/phoenix/compat/hbase/HbaseCompatCapabilities.java
@@ -18,8 +18,6 @@
 
 package org.apache.phoenix.compat.hbase;
 
-import org.apache.hadoop.conf.Configuration;
-
 public class HbaseCompatCapabilities {
 
     public static boolean isMaxLookbackTimeSupported() {

diff --git a/...e-compat-2.2.1/src/main/java/org/apache/phoenix/compat/hbase/HbaseCompatCapabilities.java b/...e-compat-2.2.1/src/main/java/org/apache/phoenix/compat/hbase/HbaseCompatCapabilities.java
@@ -18,8 +18,6 @@
 
 package org.apache.phoenix.compat.hbase;
 
-import org.apache.hadoop.conf.Configuration;
-
 public class HbaseCompatCapabilities {
 
     public static boolean isMaxLookbackTimeSupported() {
@@ -32,7 +30,9 @@ public static boolean isMaxLookbackTimeSupported() {
     //HBase 2.1 does not have HBASE-22710, which is necessary for raw scan skip scan and
     // AllVersionsIndexRebuild filters to
     // show all versions properly. HBase 2.2.5+ and HBase 2.3.0+ have this fix.
-    public static boolean isRawFilterSupported() { return true; }
+    // Since we support 2.2.1+, and this is of limited value without
+    // isLookbackBeyondDeletesSupported, we return false for all HBase 2.2.x versions
+    public static boolean isRawFilterSupported() { return false; }
 
     //HBase 2.3+ has preWALAppend() on RegionObserver (HBASE-22623)
     public static boolean hasPreWALAppend() { return false; }