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

stack commented on HBASE-19298:
-------------------------------

bq. Is it the consensus in our community? If so, I'm fine to close this issue 
as Won't Fix. 

[~chia7712] Our Sean is gone till Feb FYI as of today (For the best of reasons 
-- smile).

I'd be good w/ CellScanner being public. It is a dumb interface that is 
pervasive; e.g. Result implements it.

On the patch, whats up w/ the ExtendedCellScanner. Why we need it?

Thanks.





> CellScanner should be declared as IA.Public
> -------------------------------------------
>
>                 Key: HBASE-19298
>                 URL: https://issues.apache.org/jira/browse/HBASE-19298
>             Project: HBase
>          Issue Type: Task
>            Reporter: Chia-Ping Tsai
>            Assignee: Chia-Ping Tsai
>             Fix For: 2.0.0-beta-1
>
>         Attachments: HBASE-19298.v0.patch
>
>
> User can create the {{CellScanner}} via IA.Public {{CellUtil}}, hence 
> {{CellScanner}} should be IA.Public. However, the {{CellScanner}} is used in 
> the server code base so making {{CellScanner}} IA.Public may flaw our HBASE 
> in the future. In my opinion, we should introduce the {{ExtendedCellScanner}} 
> to replace the {{CellScanner}} for server code.



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

Reply via email to