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

stack commented on HBASE-16134:
-------------------------------

CPs could not set sequenceid w/o breakage so that is fair game to remove. 
SettableTimestamp is harder. Can see CPs doing tricks w/ this one...  You know 
how SettableTimestamp came in?

On the patch, seems odd having a Cell named InternalCell in hbase-common. 
Should it be in hbase-server? Hmm.. I see you can't do that because its used by 
 KV, etc. which is ok.  So, it should have a name like SuperCell because it is 
Cell with extra info?

We have to keep this up? boolean withTags? Can that not be an internal detail 
whether to promulgate tags?

The SuperCell does not extend Cell?

What about HeapSize, Cloneable? A SuperCell should do these too I'd say?







> Introduce InternalCell
> ----------------------
>
>                 Key: HBASE-16134
>                 URL: https://issues.apache.org/jira/browse/HBASE-16134
>             Project: HBase
>          Issue Type: Sub-task
>          Components: regionserver
>            Reporter: Anoop Sam John
>            Assignee: Anoop Sam John
>             Fix For: 2.0.0
>
>         Attachments: HBASE-16134.patch, HBASE-16134.patch
>
>
> Came after the discussion under HBASE-15721 and HBASE-15879.
> InternalCell is a Cell extension. We do have Cell extensions across different 
> interfaces now.
> SettableSeqId
> SettableTimestamp
> Streamable.
> And demand for this keep growing.
> So let us include everything into one Interface.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to