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

stack commented on HBASE-11093:
-------------------------------

bq. It would take some time before consensus is reached w.r.t. removing 
hasFilterRow().

If the better path is the removal of hasFilterRow, this is busy work.  Why not 
work on removal of hasFilterRow?  Has an argument been made (I just recall 
hackery to compensate for its misuse).

> FilterList#filterRow() iterates through its filters even though 
> FilterList#hasFilterRow() returns false
> -------------------------------------------------------------------------------------------------------
>
>                 Key: HBASE-11093
>                 URL: https://issues.apache.org/jira/browse/HBASE-11093
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Ted Yu
>            Assignee: Ted Yu
>            Priority: Minor
>         Attachments: 11093-v1.txt, 11093-v2.txt, 11093-v3.txt
>
>
> FilterList#hasFilterRow() returns false when hasFilterRow() of all of 
> constituent filters return false.
> However, FilterList#filterRow() still iterates through its filters in this 
> scenario.
> The iteration should be skipped when FilterList#hasFilterRow() returns false.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to