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

Josh Elser commented on HBASE-19092:
------------------------------------

bq. Is there any overlap between CellUtil, LimitedPrivateCellUtil and 
PrivateCellUtil? If I want to manipulate Cells I have to have import all?

For CellUtil, would it make sense to have the more-audience-restrictive 
versions extend the less-audience-restrictive verisons? e.g. 
{{LimitedPrivateCellUtil extends CellUtil}} and {{PrivateCellUtil extends 
LimitedPrivateCellUtil}}

Same for TagUtil (except we'd have to remove the final modifier on TagUtil..).

bq. Could we just add facility to add Tags to a Cell to ExtendCellBuilder (You 
know how I feel about having distinct Tag versions of things) rather than give 
it its own Builder?

I like that idea. Any reason to not make ExtendedCellBuilder LimitedPrivate 
instead of (the current) Private audience?

Alternatively, if Tag should be accessible by CPs but not things like MVCC or 
sequenceId, what about following the same pattern you did for CellUtil?

> Make Tag IA.LimitedPrivate and expose for CPs
> ---------------------------------------------
>
>                 Key: HBASE-19092
>                 URL: https://issues.apache.org/jira/browse/HBASE-19092
>             Project: HBase
>          Issue Type: Sub-task
>          Components: Coprocessors
>            Reporter: ramkrishna.s.vasudevan
>            Assignee: ramkrishna.s.vasudevan
>            Priority: Critical
>             Fix For: 2.0.0-alpha-4
>
>         Attachments: HBASE-19092-branch-2.patch, 
> HBASE-19092_001-branch-2.patch
>
>
> We need to make tags as LimitedPrivate as some use cases are trying to use 
> tags like timeline server. The same topic was discussed in dev@ and also in 
> HBASE-18995.
> Shall we target this for beta1 - cc [[email protected]].
> So once we do this all related Util methods and APIs should also move to 
> LimitedPrivate Util classes.



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

Reply via email to