[ 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)