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

Jonathan Hsieh commented on HBASE-9359:
---------------------------------------

bq. getFamily, etc., are out in CellUtil rather than in Cell currently? Their 
addition is pure convenience and bloats the Cell Interface and my fear is users 
would start to lean on these new methods rather than work harder to use the 
methods that do not require us make a copy (Matt Corgan you have an opinion 
here boss?) On other hand, smooths the transition to the Interface, a difficult 
transition to make.

I'm mostly concerned with totally borking applications (basically, all of 
them!) that depend on the old api.  I did the conversion first to catch most of 
them on the region server side, and then adding it back.

bq. bq. For the shim to handle the List<KeyValue> -> List<Cell> conversion, I'm 
going to use a naive array copy.
bq. Ouch. Put big warnings on deprecated methods that this is what is going on 
I'd say.

Oops, I didn't do the shim in v2 -- let me add the shim.
                
> Convert KeyValue to Cell in hbase-client module - Result/Put/Delete, 
> ColumnInterpreter
> --------------------------------------------------------------------------------------
>
>                 Key: HBASE-9359
>                 URL: https://issues.apache.org/jira/browse/HBASE-9359
>             Project: HBase
>          Issue Type: Sub-task
>          Components: Client
>    Affects Versions: 0.95.2
>            Reporter: Jonathan Hsieh
>            Assignee: Jonathan Hsieh
>         Attachments: hbase-9359.patch, hbase-9359.v2.patch
>
>
> This path is the second half of eliminating KeyValue from the client 
> interfaces.  This percolated through quite a bit. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to