ChiaPing Tsai commented on HBASE-16645:

hi [~anastas]

Thanks for your comments.
I have written the correct variant of this method based on more if-statements
On the review board, you said that you have written the variant, so I suppose 
you are ready to submit the patch with good readability.I apologize for any 

Address comment up on v2.patch and rb. Could you please take a look?


> Wrong range of Cells is caused by CellFlatMap#tailMap, headMap, and SubMap
> --------------------------------------------------------------------------
>                 Key: HBASE-16645
>                 URL: https://issues.apache.org/jira/browse/HBASE-16645
>             Project: HBase
>          Issue Type: Bug
>    Affects Versions: 2.0.0
>            Reporter: ChiaPing Tsai
>             Fix For: 2.0.0
>         Attachments: HBASE-16645.v0.patch, HBASE-16645.v1.patch, 
> HBASE-16645.v2.patch
> Two reasons are shown below:
> 1) CellFlatMap#find doesn’t consider desc order array
> 2) CellFlatMap#getValidIndex return the wrong upper bound

This message was sent by Atlassian JIRA

Reply via email to