[ 
https://issues.apache.org/jira/browse/HBASE-6429?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13420990#comment-13420990
 ] 

Zhihong Ted Yu commented on HBASE-6429:
---------------------------------------

For filterRow(List<KeyValue> kvs):
{code}
+    this.filter.filterRow(kvs);
+    if (!kvs.isEmpty() && this.filter.filterRow()) {
+      kvs.clear();
+    }
{code}
Can we exchange the order of filterRow(kvs) and if statement ?
If this.filter.filterRow() returns true, we don't need to call filterRow(kvs).

Correct me if I am wrong.
                
> Filter with filterRow() returning true is incompatible with scan with limit
> ---------------------------------------------------------------------------
>
>                 Key: HBASE-6429
>                 URL: https://issues.apache.org/jira/browse/HBASE-6429
>             Project: HBase
>          Issue Type: Bug
>          Components: filters
>    Affects Versions: 0.96.0
>            Reporter: Jason Dai
>         Attachments: hbase-6429-trunk.patch, hbase-6429_0_94_0.patch
>
>
> Currently if we scan with bot limit and a Filter with 
> filterRow(List<KeyValue>) implemented, an  IncompatibleFilterException will 
> be thrown. The same exception should also be thrown if the filer has its 
> filterRow() implemented.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to