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

Anoop Sam John commented on HBASE-13387:
----------------------------------------

This will be a big patch. So I am planning to split into multiple tasks and 
make core code ready for accepting BB backed cells.
- Make Tag as an interface impl way
- Deperecate filterRowKey(byte[] buffer, int offset, int length) in favor of 
filterRowKey(Cell firstRowCell)
- Deprecate postScannerFilterRow CP hook with byte[],int,int args in favor of 
taking Cell arg
- Change ColumnTracker methods to pass Cell instead of byte[], int, int for 
column.
- Remove CellComparator#compareRows(byte[], int, int, byte[], int,int)
- CellUtil  - More typed getters from Cell components (Like getRowAsInt, 
getValueAsLong etc)
.....

> Add ByteBufferedCell an extension to Cell
> -----------------------------------------
>
>                 Key: HBASE-13387
>                 URL: https://issues.apache.org/jira/browse/HBASE-13387
>             Project: HBase
>          Issue Type: Sub-task
>          Components: regionserver, Scanners
>            Reporter: Anoop Sam John
>            Assignee: Anoop Sam John
>         Attachments: ByteBufferedCell.docx, WIP_HBASE-13387_V2.patch, 
> WIP_ServerCell.patch
>
>
> This came in btw the discussion abt the parent Jira and recently Stack added 
> as a comment on the E2E patch on the parent Jira.
> The idea is to add a new Interface 'ByteBufferedCell'  in which we can add 
> new buffer based getter APIs and getters for position in components in BB.  
> We will keep this interface @InterfaceAudience.Private.   When the Cell is 
> backed by a DBB, we can create an Object implementing this new interface.
> The Comparators has to be aware abt this new Cell extension and has to use 
> the BB based APIs rather than getXXXArray().  Also give util APIs in CellUtil 
> to abstract the checks for new Cell type.  (Like matchingXXX APIs, 
> getValueAs<type> APIs etc)



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

Reply via email to