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