[ https://issues.apache.org/jira/browse/HBASE-14398?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14947221#comment-14947221 ]
stack commented on HBASE-14398: ------------------------------- Thanks Anoop. So, BBC will behind the scenes just copy from offheap to onheap if you use the array methods from Cell. I remember us agreeing that this is how this should work. So, Cells 'length' methods -- getXXXLength -- works for BBC and for Cell. The discussion here is about the offset argument. The getXXXOffset will return one value if backed by an array and another if by an offheap BB. I reread the BBC class comment. It is good on how the Interface should be used. Let me add a patch with some more given the discussion above so you don't have to re-explain it all to me again (you and Ram). So, now, what to call the 'offset' method when a BBC? getXXXPosition? The BB doc talks about position (though it returns it in a method called arrayOffset).? bq. Can this be getXXXBufferOffset()? Other methods in BBC talk of ByteBuffer. The above shortens it to Buffer only. Let me make a patch that folds this helpful feedback from you fellows into javadoc on BBC. Thanks. > Create the fake keys required in the scan path to avoid copy to byte[] > ---------------------------------------------------------------------- > > Key: HBASE-14398 > URL: https://issues.apache.org/jira/browse/HBASE-14398 > Project: HBase > Issue Type: Sub-task > Reporter: ramkrishna.s.vasudevan > Assignee: ramkrishna.s.vasudevan > Fix For: 2.0.0 > > Attachments: HBASE-14398.patch, HBASE-14398_1.patch > > > Already we have created some fake keys for the ByteBufferedCells so that we > can avoid the copy requried to create fake keys. This JIRA aims to fill up > all such places so that the Offheap BBs are not copied to onheap byte[]. -- This message was sent by Atlassian JIRA (v6.3.4#6332)