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

Chia-Ping Tsai commented on HBASE-19027:
----------------------------------------

bq. Please add javadoc for the new parameter.
copy that.

bq. Will scan info always have a comparator? It looks like yes, but I didn't 
follow through on all the code paths. Should we default back to the store 
comparator if there isn't one.
The matcher in {{StoreScanner}} uses the {{CellComparator}} from {{ScanInfo}}, 
hence the {{StoreScanner}} should do the same as matcher.

> Honor the CellComparator of ScanInfo in scanning over a store
> -------------------------------------------------------------
>
>                 Key: HBASE-19027
>                 URL: https://issues.apache.org/jira/browse/HBASE-19027
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Chia-Ping Tsai
>            Assignee: Chia-Ping Tsai
>            Priority: Major
>             Fix For: 2.0.0-beta-1
>
>         Attachments: HBASE-19027.v0.patch, HBASE-19027.v1.patch
>
>
> All components in the scan path should honor the cell comparator from 
> scaninfo rather than arbitrarily choose the comparator from store, scaninfo, 
> or static COMPARATOR.



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

Reply via email to