You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@hbase.apache.org by "Duo Zhang (JIRA)" <ji...@apache.org> on 2016/10/14 10:18:20 UTC

[jira] [Commented] (HBASE-16840) Reuse cell's timestamp and type in ScanQueryMatcher

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

Duo Zhang commented on HBASE-16840:
-----------------------------------

Have you tested the performance impact of the patch?

The patch makes the code a bit confusing as you do not assign timestamp and typeByte at the first place so later people may wonder whether it is safe to use these fields...

> Reuse cell's timestamp and type in ScanQueryMatcher
> ---------------------------------------------------
>
>                 Key: HBASE-16840
>                 URL: https://issues.apache.org/jira/browse/HBASE-16840
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: binlijin
>         Attachments: HBASE-16840_master.patch
>
>
> Reuse cell's timestamp and type in ScanQueryMatcher, this is useful for KeyValue.



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