You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Sean Busbey (JIRA)" <ji...@apache.org> on 2017/11/18 18:33:00 UTC
[jira] [Commented] (HBASE-19298) CellScanner should be declared as
IA.Public
[ https://issues.apache.org/jira/browse/HBASE-19298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16258141#comment-16258141 ]
Sean Busbey commented on HBASE-19298:
-------------------------------------
There are plenty of places where our IA.Public API uses or returns non-IA.Public members that folks are advised to treat as a black box. what's special about CellScanner that we should make it public?
> 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)