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

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

bq. But one thing is that now as part of this patch we are now making 
ExtendedCellBuilder CP exposed. So we ideally need RawCellBuilder - because 
that is where Tags are coming? Can we focus all that part in HBASE-19112 then?

Makes sense (Returning a RawCellBuilder sounds good. In it you would not allow 
option for setting sequenceid?). Sounds good as part of HBASE-19112. Skimming 
this patch, the getType belongs in RawCell going by rationale in HBASE-19112. 
Otherwise, it looks good to me. Yeah, need to fix CpEnv getCellBuilder. Good 
stuff [~ram_krish] LMK. I could do these little changes on top of your patch 
sir.

> 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-branch-2_5.patch, HBASE-19092-branch-2_5.patch, 
> HBASE-19092.branch-2.0.02.patch, HBASE-19092_001-branch-2.patch, 
> HBASE-19092_001.patch, HBASE-19092_002-branch-2.patch, HBASE-19092_002.patch, 
> HBASE-19092_004.patch, HBASE-19092_3.patch, HBASE-19092_4.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