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

ramkrishna.s.vasudevan commented on HBASE-10531:
------------------------------------------------

I think for now we need to add Cell comparison methods in KVComparator also, 
because we have lot of code internally that references this Kvcomparator and 
calls comparator.compareXXX().
So I would suggest we could make the change in using a CellComparator in 
another JIRA.  If not for now I will do this
{code}
protected int compareRowKey(final Cell left, final Cell right) {
      CellComparator.compareRows(left, right);
    }
{code}

> Revisit how the key byte[] is passed to HFileScanner.seekTo and reseekTo
> ------------------------------------------------------------------------
>
>                 Key: HBASE-10531
>                 URL: https://issues.apache.org/jira/browse/HBASE-10531
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: ramkrishna.s.vasudevan
>            Assignee: ramkrishna.s.vasudevan
>             Fix For: 0.99.0
>
>         Attachments: HBASE-10531.patch, HBASE-10531_1.patch, 
> HBASE-10531_2.patch
>
>
> Currently the byte[] key passed to HFileScanner.seekTo and 
> HFileScanner.reseekTo, is a combination of row, cf, qual, type and ts.  And 
> the caller forms this by using kv.getBuffer, which is actually deprecated.  
> So see how this can be achieved considering kv.getBuffer is removed.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to