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

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

bq.We have it in Region, Store, StoreFile etc.
Reading the code I think for having Comparator in Store or not depends on what 
type of InternalScanner are we going to expose. How ever if the new interface 
is only exposing some restrcited but powerful APIs then it is still fine to 
expose in Store? So that CPs in case they need to filter out or do some 
processing on KVs can do it at the InternalScanner level?
StoreFile level we may not need it IMHO.
bq.Should we purge the Serializable if we prepare to make CellComparator 
IA.Public?
Is there any harm in having it Serializable? Am not very sure on this.

> Make a Public interface for CellComparator
> ------------------------------------------
>
>                 Key: HBASE-18945
>                 URL: https://issues.apache.org/jira/browse/HBASE-18945
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: ramkrishna.s.vasudevan
>            Assignee: ramkrishna.s.vasudevan
>             Fix For: 2.0.0-beta-1
>
>
> Based on discussions over in HBASE-18826 and HBASE-18183 it is better we 
> expose CellComparator as a public interface so that it could be used in 
> Region/Store interfaces to be exposed to CPs.



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

Reply via email to