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

ramkrishna.s.vasudevan commented on HBASE-13122:
------------------------------------------------

{code}
     if (familyLength > 0) {
       if (doCompare(this.compareOp, this.comparator, v.getFamilyArray(),
           v.getFamilyOffset(), familyLength)) {
-        return ReturnCode.SKIP;
+        return ReturnCode.NEXT_ROW;
       }
     }
{code}
I doubt this change.  Suppose I have 2 families and the first family is what I 
want to filter then how can I move to the next_row? 
In the tests that was done 
bq.scan use familyFilter get values from the second family, scaned 2000 rows 
and 100 versions of each row& qualifier.
Use the oraginal FamilyFilter, cost average 309 seconds, but with the improved 
familyFilter, the cost is average 38 seconds, improved about 700%
The values were got from the secondFamily so that would not be a problem. But 
if it had been the first family we would have missed the cells?  Pls correct me 
if am wrong here.

> Improve efficiency for return codes of some filters
> ---------------------------------------------------
>
>                 Key: HBASE-13122
>                 URL: https://issues.apache.org/jira/browse/HBASE-13122
>             Project: HBase
>          Issue Type: Improvement
>          Components: Filters
>    Affects Versions: 0.94.24, 1.0.1, 0.98.10.1
>            Reporter: Shuaifeng Zhou
>             Fix For: 2.0.0, 1.1.0
>
>         Attachments: 13122-master.patch, 13122.patch
>
>
> ColumnRangeFilter:
>  when minColumnInclusive is false, it means all the cells at the current 
> row&column not fit the condition, so it should skip to next column, return 
> code should be NEXT_COL, not SKIP.
> FamilyFilter is the similar sitution.
> Currently, SKIP will not causing error, but not efficent.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to