HBASE-22262 Removed deprecated methods from Filter class

Author: HorizonNet

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/ColumnCountGetFilter.java

@@ -63,12 +63,6 @@ public boolean filterAllRemaining() {
     return this.count > this.limit;
   }
 
-  @Deprecated
-  @Override
-  public ReturnCode filterKeyValue(final Cell c) {
-    return filterCell(c);
-  }
-
   @Override
   public ReturnCode filterCell(final Cell c) {
     this.count++;

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/ColumnPaginationFilter.java

@@ -111,12 +111,6 @@ public boolean filterRowKey(Cell cell) throws IOException {
     return false;
   }
 
-  @Override
-  @Deprecated
-  public ReturnCode filterKeyValue(final Cell c) {
-    return filterCell(c);
-  }
-
   @Override
   public ReturnCode filterCell(final Cell c)
   {

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/ColumnPrefixFilter.java

@@ -58,12 +58,6 @@ public boolean filterRowKey(Cell cell) throws IOException {
     return false;
   }
 
-  @Deprecated
-  @Override
-  public ReturnCode filterKeyValue(final Cell c) {
-    return filterCell(c);
-  }
-
   @Override
   public ReturnCode filterCell(final Cell cell) {
     if (this.prefix == null) {

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/ColumnRangeFilter.java

@@ -122,12 +122,6 @@ public boolean filterRowKey(Cell cell) throws IOException {
     return false;
   }
 
-  @Override
-  @Deprecated
-  public ReturnCode filterKeyValue(final Cell c) {
-    return filterCell(c);
-  }
-
   @Override
   public ReturnCode filterCell(final Cell c) {
     int cmpMin = 1;

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/DependentColumnFilter.java

@@ -133,12 +133,6 @@ public boolean filterAllRemaining() {
     return false;
   }
 
-  @Deprecated
-  @Override
-  public ReturnCode filterKeyValue(final Cell c) {
-    return filterCell(c);
-  }
-
   @Override
   public ReturnCode filterCell(final Cell c) {
     // Check if the column and qualifier match
@@ -173,10 +167,6 @@ public boolean filterRow() {
     return false;
   }
 
-  @Override
-  public boolean filterRowKey(byte[] buffer, int offset, int length) {
-    return false;
-  }
   @Override
   public void reset() {
     stampSet.clear();    

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/FamilyFilter.java

@@ -56,12 +56,6 @@ public FamilyFilter(final CompareOperator op,
     super(op, familyComparator);
   }
 
-  @Deprecated
-  @Override
-  public ReturnCode filterKeyValue(final Cell c) {
-    return filterCell(c);
-  }
-
   @Override
   public ReturnCode filterCell(final Cell c) {
     int familyLength = c.getFamilyLength();

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/Filter.java

@@ -64,24 +64,6 @@ public abstract class Filter {
    */
   abstract public void reset() throws IOException;
 
-  /**
-   * Filters a row based on the row key. If this returns true, the entire row will be excluded. If
-   * false, each KeyValue in the row will be passed to {@link #filterCell(Cell)} below.
-   * 
-   * Concrete implementers can signal a failure condition in their code by throwing an
-   * {@link IOException}.
-   * 
-   * @param buffer buffer containing row key
-   * @param offset offset into buffer where row key starts
-   * @param length length of the row key
-   * @return true, remove entire row, false, include the row (maybe).
-   * @throws IOException in case an I/O or an filter specific failure needs to be signaled.
-   * @deprecated As of release 2.0.0, this will be removed in HBase 3.0.0.
-   *             Instead use {@link #filterRowKey(Cell)}
-   */
-  @Deprecated
-  abstract public boolean filterRowKey(byte[] buffer, int offset, int length) throws IOException;
-
   /**
    * Filters a row based on the row key. If this returns true, the entire row will be excluded. If
    * false, each KeyValue in the row will be passed to {@link #filterCell(Cell)} below.
@@ -108,34 +90,6 @@ public abstract class Filter {
    */
   abstract public boolean filterAllRemaining() throws IOException;
 
-  /**
-   * A way to filter based on the column family, column qualifier and/or the column value. Return
-   * code is described below. This allows filters to filter only certain number of columns, then
-   * terminate without matching ever column.
-   * 
-   * If filterRowKey returns true, filterKeyValue needs to be consistent with it.
-   * 
-   * filterKeyValue can assume that filterRowKey has already been called for the row.
-   * 
-   * If your filter returns <code>ReturnCode.NEXT_ROW</code>, it should return
-   * <code>ReturnCode.NEXT_ROW</code> until {@link #reset()} is called just in case the caller calls
-   * for the next row.
-   *
-   * Concrete implementers can signal a failure condition in their code by throwing an
-   * {@link IOException}.
-   * 
-   * @param c the Cell in question
-   * @return code as described below, Filter.ReturnCode.INCLUDE by default
-   * @throws IOException in case an I/O or an filter specific failure needs to be signaled.
-   * @see Filter.ReturnCode
-   * @deprecated As of release 2.0.0, this will be removed in HBase 3.0.0.
-   *             Instead use filterCell(Cell)
-   */
-  @Deprecated
-  public ReturnCode filterKeyValue(final Cell c) throws IOException {
-    return Filter.ReturnCode.INCLUDE;
-  }
-
   /**
    * A way to filter based on the column family, column qualifier and/or the column value. Return
    * code is described below. This allows filters to filter only certain number of columns, then
@@ -157,8 +111,8 @@ public ReturnCode filterKeyValue(final Cell c) throws IOException {
    * @throws IOException in case an I/O or an filter specific failure needs to be signaled.
    * @see Filter.ReturnCode
    */
-  public ReturnCode filterCell(final Cell c) throws IOException{
-    return filterKeyValue(c);
+  public ReturnCode filterCell(final Cell c) throws IOException {
+    return ReturnCode.INCLUDE;
   }
 
   /**

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/FilterBase.java

@@ -47,25 +47,9 @@ public abstract class FilterBase extends Filter {
   public void reset() throws IOException {
   }
 
-  /**
-   * Filters that do not filter by row key can inherit this implementation that
-   * never filters anything. (ie: returns false).
-   *
-   * {@inheritDoc}
-   * @deprecated As of release 2.0.0, this will be removed in HBase 3.0.0.
-   *             Instead use {@link #filterRowKey(Cell)}
-   */
-  @Override
-  @Deprecated
-  public boolean filterRowKey(byte[] buffer, int offset, int length) throws IOException {
-    if (filterAllRemaining()) return true;
-    return false;
-  }
-
   @Override
   public boolean filterRowKey(Cell cell) throws IOException {
-    if (filterAllRemaining()) return true;
-    return filterRowKey(cell.getRowArray(), cell.getRowOffset(), cell.getRowLength());
+    return filterAllRemaining();
   }
 
   /**

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/FilterList.java

@@ -146,11 +146,6 @@ public void reset() throws IOException {
     filterListBase.reset();
   }
 
-  @Override
-  public boolean filterRowKey(byte[] rowKey, int offset, int length) throws IOException {
-    return filterListBase.filterRowKey(rowKey, offset, length);
-  }
-
   @Override
   public boolean filterRowKey(Cell firstRowCell) throws IOException {
     return filterListBase.filterRowKey(firstRowCell);
@@ -166,12 +161,6 @@ public Cell transformCell(Cell c) throws IOException {
     return filterListBase.transformCell(c);
   }
 
-  @Override
-  @Deprecated
-  public ReturnCode filterKeyValue(final Cell c) throws IOException {
-    return filterCell(c);
-  }
-
   @Override
   public ReturnCode filterCell(final Cell c) throws IOException {
     return filterListBase.filterCell(c);

hbase-client/src/main/java/org/apache/hadoop/hbase/filter/FilterListBase.java

@@ -112,11 +112,6 @@ public Cell transformCell(Cell c) throws IOException {
     return transformed;
   }
 
-  @Override
-  public ReturnCode filterKeyValue(final Cell c) throws IOException {
-    return filterCell(c);
-  }
-
   /**
    * Filters that never filter by modifying the returned List of Cells can inherit this
    * implementation that does nothing. {@inheritDoc}