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

Anoop Sam John commented on HBASE-18904:
----------------------------------------

This a return statement in the if block so after that break is there or not 
matters?  Ya above cases break is there but here missing. Still that will give 
functional issue?

> Missing break in NEXT_ROW case of FilterList#mergeReturnCodeForOrOperator()
> ---------------------------------------------------------------------------
>
>                 Key: HBASE-18904
>                 URL: https://issues.apache.org/jira/browse/HBASE-18904
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Ted Yu
>
> Here is related code, around line 569:
> {code}
>       if (isInReturnCodes(rc, ReturnCode.NEXT_ROW)) {
>         return ReturnCode.NEXT_ROW;
>       }
>     case SEEK_NEXT_USING_HINT:
> {code}
> break was missing for the NEXT_ROW case.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to