You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by la...@apache.org on 2012/11/05 01:55:22 UTC

svn commit: r1405684 - /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/filter/KeyOnlyFilter.java

Author: larsh
Date: Mon Nov  5 00:55:21 2012
New Revision: 1405684

URL: http://svn.apache.org/viewvc?rev=1405684&view=rev
Log:
HBASE-7095 Cannot set 'lenAsVal' for KeyOnlyFilter from shell (Aditya Kishore)

Modified:
    hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/filter/KeyOnlyFilter.java

Modified: hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/filter/KeyOnlyFilter.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/filter/KeyOnlyFilter.java?rev=1405684&r1=1405683&r2=1405684&view=diff
==============================================================================
--- hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/filter/KeyOnlyFilter.java (original)
+++ hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/filter/KeyOnlyFilter.java Mon Nov  5 00:55:21 2012
@@ -48,9 +48,13 @@ public class KeyOnlyFilter extends Filte
   }
 
   public static Filter createFilterFromArguments(ArrayList<byte []> filterArguments) {
-    Preconditions.checkArgument(filterArguments.size() == 0,
-                                "Expected: 0 but got: %s", filterArguments.size());
-    return new KeyOnlyFilter();
+    Preconditions.checkArgument((filterArguments.size() == 0 || filterArguments.size() == 1),
+                                "Expected: 0 or 1 but got: %s", filterArguments.size());
+    KeyOnlyFilter filter = new KeyOnlyFilter();
+    if (filterArguments.size() == 1) {
+      filter.lenAsVal = ParseFilter.convertByteArrayToBoolean(filterArguments.get(0));
+    }
+    return filter;
   }
 
   public void write(DataOutput out) throws IOException {