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

Hudson commented on HBASE-13998:
--------------------------------

SUCCESS: Integrated in HBase-TRUNK #6627 (See 
[https://builds.apache.org/job/HBase-TRUNK/6627/])
HBASE-13998 Remove CellComparator#compareRows(byte[] left, int loffset, int 
llength, byte[] right, int roffset, int rlength)- Addendum to fix javadoc warn 
(anoopsamjohn: rev 215290eb76925738550648b2ba9e800ccb7cf0da)
* hbase-common/src/main/java/org/apache/hadoop/hbase/CellComparator.java


> Remove CellComparator#compareRows(byte[] left, int loffset, int llength, 
> byte[] right, int roffset, int rlength)
> ----------------------------------------------------------------------------------------------------------------
>
>                 Key: HBASE-13998
>                 URL: https://issues.apache.org/jira/browse/HBASE-13998
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Anoop Sam John
>            Assignee: Anoop Sam John
>             Fix For: 2.0.0
>
>         Attachments: HBASE-13998.patch, HBASE-13998_addendum.patch
>
>
> A public API in CellComparator which takes old style byte[], offset, length 
> alone is not correct.  CellComparator supposed to compare cell(s).  At least 
> one side param has to be a cell.. This is the agreement we discussed in 
> HBASE-10800.  Still we could not remove the above one method because it was 
> getting used from multiple places.  Now most of the usage is removed.  This 
> jira aims at removing it fully and replace the usage with other APIs.
> Note: The CellComparator is added in 2.0 only so removing the public API is 
> not creating any BC issue.



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

Reply via email to