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

stack commented on HBASE-19092:
-------------------------------

bq. I think instead of LPTagUtil we could add all the static methods to Tag 
interface itself. Tag will be marked LP for CPs.

Would this include means of building Tags?

bq. As per Stack's suggestion we can add a getCellBuilder in the RegionCPEnv. 
So users can pass the type of CellBuilder and that will return an 
ExtendedCellBulder and that will be made LP. 

This is easy to do and we can shutdown access to expose only what we want CPs 
to see.

bq. The only catch is that people can do setSeqID on that. We should avoid that 
may be in beta-1 or later.

We can figure this one I'd say.

I'm not following why the TagsUtil. What would be in the Tags Interface as 
static helpers? Would there be a Tags builder/factory as we have for Cell or 
would the Tags building be done in Cell (this latter seems most 'natural').

Ok... work to do here still. Punting this to beta-1. Let me try and help out 
there. Thanks lads.





> 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-beta-1
>
>         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