[ https://issues.apache.org/jira/browse/HBASE-9383?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13904809#comment-13904809 ]
Enis Soztutar commented on HBASE-9383: -------------------------------------- Agreed with Stack's comments on HBASE-7320. I don't think HeapSize belongs to the Cell interface. Maybe we can have a CellContainer interface, where CellContainer, and KV and encoders implement CellContainer. Also, since Cell is a public facing API, I think we have to be careful to not include methods. We actually want to hide the internal methods in Cell. > Have Cell interface extend HeapSize interface > --------------------------------------------- > > Key: HBASE-9383 > URL: https://issues.apache.org/jira/browse/HBASE-9383 > Project: HBase > Issue Type: Sub-task > Affects Versions: 0.96.0 > Reporter: Jonathan Hsieh > > From the review of HBASE-9359. > bq. Stack: Cell should implement HeapSize? That sounds right. > bq. Ram: +1 for Cell extending HeapSize. > bq. Jon: I'll look into Cell extending HeapSize as a follow on. It doesn't > interfere with the signature and if a older 0.96 client talks to a 0.96.1 rs > that has the change, it won't matter. -- This message was sent by Atlassian JIRA (v6.1.5#6160)