HBASE-28518 Allow specifying a filter for the REST multiget endpoint …

…(addendum: revert SCAN_FILTER constant name)
apache · Apr 25, 2024 · 5b739e6 · 5b739e6
1 parent 6c6e776
commit 5b739e6
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/Constants.java b/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/Constants.java
@@ -86,6 +86,11 @@ public interface Constants {
   String SCAN_LIMIT = "limit";
   String SCAN_FETCH_SIZE = "hbase.rest.scan.fetchsize";
   String FILTER = "filter";
+  /**
+   * @deprecated Since 2.4.18/2.5.9/2.6.0/3.0/4.0. Use {@link #FILTER}
+   */
+  @Deprecated
+  String SCAN_FILTER = FILTER;
   String FILTER_B64 = "filter_b64";
   String SCAN_REVERSED = "reversed";
   String SCAN_CACHE_BLOCKS = "cacheblocks";

diff --git a/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/MultiRowResource.java b/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/MultiRowResource.java
@@ -87,7 +87,6 @@ public Response get(final @Context UriInfo uriInfo,
     if (paramFilterB64 != null) {
       filterBytes = base64Urldecoder.decode(paramFilterB64);
     } else if (paramFilter != null) {
-      // Not binary clean
       filterBytes = paramFilter.getBytes();
     }
 

diff --git a/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/TableResource.java b/hbase-rest/src/main/java/org/apache/hadoop/hbase/rest/TableResource.java
@@ -183,7 +183,6 @@ public TableScanResource getScanResource(final @PathParam("scanspec") String sca
       if (paramFilterB64 != null) {
         filterBytes = base64Urldecoder.decode(paramFilterB64);
       } else if (paramFilter != null) {
-        // Not binary clean
         filterBytes = paramFilter.getBytes();
       }
       if (filterBytes != null) {