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

ramkrishna.s.vasudevan commented on HBASE-19112:
------------------------------------------------

[~chia7712]
Sorry for being late here. I checked the code. 
Cell#getTypeByte() is used in the server side and in the PBUtil class to 
convert the mutations to respective cells and vice versa.
So I think we can safely assume the Cells in this PBUtil APIs are of type 
RawCell? Because from RPC layer we decode the cells and create KVs or BBKVs.
The other way also that will be true.
Now I can see that already we have marked getTypeByte() as deprecated and 
pointed to use CellUtil API. Ideally if the cells that are going to be returned 
back or going to be of type Put only then we don need any such API in Cell. How 
ever if a client needs to verify that it has created only Delete type of cells 
then still CellUtil API can be used. 

> Suspect methods on Cell to be deprecated
> ----------------------------------------
>
>                 Key: HBASE-19112
>                 URL: https://issues.apache.org/jira/browse/HBASE-19112
>             Project: HBase
>          Issue Type: Bug
>          Components: Client
>            Reporter: Josh Elser
>            Assignee: ramkrishna.s.vasudevan
>            Priority: Blocker
>             Fix For: 2.0.0-beta-1
>
>         Attachments: HBASE-19112_branch-2.patch
>
>
> [~chia7712] suggested on the [mailing 
> list|https://lists.apache.org/thread.html/e6de9af26d9b888a358ba48bf74655ccd893573087c032c0fcf01585@%3Cdev.hbase.apache.org%3E]
>  that we have some methods on Cell which should be deprecated for removal:
> * {{#getType()}}
> * {{#getTimestamp()}}
> * {{#getTag()}}
> * {{#getSequenceId()}}
> Let's make a pass over these (and maybe the rest) to make sure that there 
> aren't others which are either implementation details or methods returning 
> now-private-marked classes.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to