Updated usage of filterRowKey() to align with HBase 3 API changes

Author: szucsvillo

phoenix-core-client/src/main/java/org/apache/phoenix/filter/MultiKeyValueComparisonFilter.java

@@ -200,7 +200,7 @@ public Void visit(KeyValueColumnExpression expression) {
         expression.accept(visitor);
     }
 
-    @Override
+    // No @Override for Hbase 3 compatibility
     public ReturnCode filterKeyValue(Cell cell) {
         return filterCell(cell);
     }

phoenix-core-client/src/main/java/org/apache/phoenix/filter/SkipScanFilter.java

@@ -166,7 +166,7 @@ public boolean filterAllRemaining() {
         return isDone;
     }
 
-    @Override
+    // No @Override for HBase 3 compatibility
     public ReturnCode filterKeyValue(Cell kv) {
         return filterCell(kv);
     }

phoenix-core-client/src/main/java/org/apache/phoenix/filter/SystemCatalogViewIndexIdFilter.java

@@ -54,7 +54,7 @@ public SystemCatalogViewIndexIdFilter(int clientVersion) {
         this.clientVersion = clientVersion;
     }
 
-    @Override
+    // No @Override for HBase 3 compatibility
     public ReturnCode filterKeyValue(Cell keyValue) {
         return filterCell(keyValue);
     }

phoenix-core-client/src/main/java/org/apache/phoenix/hbase/index/covered/filter/ApplyAndFilterDeletesFilter.java

@@ -102,7 +102,7 @@ public Cell getNextCellHint(Cell peeked){
     return currentHint.getHint(PhoenixKeyValueUtil.maybeCopyCell(peeked));
   }
 
-  @Override
+  // No @Override for HBase 3 compatibility
   public ReturnCode filterKeyValue(Cell next) {
     return this.filterCell(next);
   }

phoenix-core-client/src/main/java/org/apache/phoenix/hbase/index/covered/filter/ColumnTrackingNextLargestTimestampFilter.java

@@ -47,7 +47,7 @@ public ColumnTrackingNextLargestTimestampFilter(long maxTime, ColumnTracker toTr
     this.column = toTrack;
   }
 
-  @Override
+  // No @Override for HBase 3 compatibility
   public ReturnCode filterKeyValue(Cell v) {
     return this.filterCell(v);
   }

phoenix-core-client/src/main/java/org/apache/phoenix/hbase/index/covered/filter/MaxTimestampFilter.java

@@ -51,7 +51,7 @@ public Cell getNextCellHint(Cell currentKV) {
     return kv;
   }
 
-  @Override
+  // No @Override for HBase 3 compatibility
   public ReturnCode filterKeyValue(Cell v) {
     return this.filterCell(v);
   }

phoenix-core-client/src/main/java/org/apache/phoenix/hbase/index/covered/filter/NewerTimestampFilter.java

@@ -34,7 +34,7 @@ public NewerTimestampFilter(long timestamp) {
     this.timestamp = timestamp;
   }
 
-  @Override
+  // No @Override for HBase 3 compatibility
   public ReturnCode filterKeyValue(Cell ignored) {
     return this.filterCell(ignored);
   }

phoenix-core-server/src/main/java/org/apache/phoenix/coprocessor/SuffixFilter.java

@@ -646,7 +646,7 @@ public SeekNext(byte[] rowkey, byte[] hint) {
             KeyValue kv = KeyValueUtil.createFirstOnRow(rowkey);
             skipper.reset();
             assertFalse(skipper.filterAllRemaining());
-            assertFalse(skipper.filterRowKey(kv.getBuffer(), kv.getRowOffset(), kv.getRowLength()));
+            assertFalse(skipper.filterRowKey(kv));
 
             assertEquals(ReturnCode.SEEK_NEXT_USING_HINT, skipper.filterCell(kv));
             assertEquals(KeyValueUtil.createFirstOnRow(hint), skipper.getNextCellHint(kv));
@@ -672,7 +672,7 @@ public Include(byte[] rowkey) {
             KeyValue kv = KeyValueUtil.createFirstOnRow(rowkey);
             skipper.reset();
             assertFalse(skipper.filterAllRemaining());
-            assertFalse(skipper.filterRowKey(kv.getBuffer(), kv.getRowOffset(), kv.getRowLength()));
+            assertFalse(skipper.filterRowKey(kv));
             assertEquals(kv.toString(), ReturnCode.INCLUDE_AND_NEXT_COL, skipper.filterCell(kv));
         }