hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "stack (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-11027) Remove kv.isDeleteXX() and related methods and use CellUtil apis.
Date Tue, 24 Jun 2014 21:12:25 GMT

    [ 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)

Mime
View raw message