[ 
https://issues.apache.org/jira/browse/HBASE-13998?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Anoop Sam John updated HBASE-13998:
-----------------------------------
      Resolution: Fixed
    Hadoop Flags: Reviewed
          Status: Resolved  (was: Patch Available)

Added the code comments in MetaCache.
Thanks for the reviews Ram and Stack.

> 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
>
>
> 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