You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Anoop Sam John (JIRA)" <ji...@apache.org> on 2016/02/11 07:02:18 UTC

[jira] [Commented] (HBASE-15253) Small bug in CellUtil.matchingRow(Cell, byte[])

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

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

Ideally we will never call it with null 'buf'..  Still +1 to fix it.

> Small bug in CellUtil.matchingRow(Cell, byte[])
> -----------------------------------------------
>
>                 Key: HBASE-15253
>                 URL: https://issues.apache.org/jira/browse/HBASE-15253
>             Project: HBase
>          Issue Type: Bug
>            Reporter: ramkrishna.s.vasudevan
>            Assignee: ramkrishna.s.vasudevan
>             Fix For: 2.0.0
>
>
> {code}
>   public static boolean matchingRow(final Cell left, final byte[] buf) {
>     if (buf == null) {
>       return left.getQualifierLength() == 0;
>     }
>     return matchingRow(left, buf, 0, buf.length);
>   }
> {code}
> The 'if' condition should check for left.getRowLength() == 0.



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