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

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

bq.we can add a new static helper method to CellUtil that compares everything.
+1 for this.. Lets not change any thing in CellComparator (IMHO)

> CellComparator compare function should compare cell's value along with other 
> attributes
> ---------------------------------------------------------------------------------------
>
>                 Key: HBASE-18361
>                 URL: https://issues.apache.org/jira/browse/HBASE-18361
>             Project: HBase
>          Issue Type: Bug
>    Affects Versions: 1.3.1
>            Reporter: Amit Kabra
>            Assignee: Amit Kabra
>         Attachments: HBASE-18361.patch
>
>
> CellComparator.compare(Cell a, Cell b) should compare a and b's value as well.
> If we create two cells as 
>     byte [] row = Bytes.toBytes("row");
>     byte [] value = Bytes.toBytes("value");
>     byte [] value1 = Bytes.toBytes("value1");
>     Cell c1 = CellUtil.createCell(row, value);
>     Cell c2 = CellUtil.createCell(row, value1);
> And compare them using CellComparator.compare(c1, c2,true) , it returns 0 i.e 
> it matches them while they are different.CellComparator compares each 
> attribute of cell but value.



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

Reply via email to