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

stack commented on HBASE-11027:
-------------------------------

This is great.

Have isDelete(final Cell cell) { call public static boolean isDelete(final byte 
type) { if you make a new patch?  Can do it on commit.

I suppose removing the methods from KV is ok since it is marked Private 
audience.

+1

> Remove kv.isDeleteXX() and related methods and use CellUtil apis.
> -----------------------------------------------------------------
>
>                 Key: HBASE-11027
>                 URL: https://issues.apache.org/jira/browse/HBASE-11027
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: ramkrishna.s.vasudevan
>            Assignee: ramkrishna.s.vasudevan
>             Fix For: 0.99.0
>
>         Attachments: HBASE-11027.patch
>
>
> WE have code like 
> {code}
> kv.isLatestTimestamp() && kv.isDeleteType()
> {code}
> We could remove them and use CellUtil.isDeleteType() so that Cells can be 
> directly used instead of Converting a cell to kv.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to