[jira] [Commented] (HBASE-16665) Check whether KeyValueUtil.createXXX could be replaced by CellUtil without copy

[Heng Chen (JIRA)]

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

Heng Chen commented on HBASE-16665:
-----------------------------------

Not sure for the two place whether KeyValueUtil.createFirstOnRow could be 
replaced. 

{code: title=AbstractMemStore.java}
    Cell firstCell = KeyValueUtil.createFirstOnRow(
        cell.getRowArray(), cell.getRowOffset(), cell.getRowLength(),
        cell.getFamilyArray(), cell.getFamilyOffset(), cell.getFamilyLength(),
        cell.getQualifierArray(), cell.getQualifierOffset(), 
cell.getQualifierLength());
    SortedSet<Cell> ss = active.tailSet(firstCell);
{code}

{code: title=ScanQueryMatcher.java}
    KeyValue kv = KeyValueUtil.createFirstOnRow(HConstants.EMPTY_BYTE_ARRAY, 0, 
0,
      HConstants.EMPTY_BYTE_ARRAY, 0, 0, bytes, offset, length);
    MatchCode matchCode = columnTracker.checkColumn(kv, type);
    if (matchCode == MatchCode.INCLUDE) {
      return columnTracker.checkVersions(kv, ttl, type, ignoreCount);
    }
    return matchCode;
{code}

> Check whether KeyValueUtil.createXXX could be replaced by CellUtil without 
> copy
> -------------------------------------------------------------------------------
>
>                 Key: HBASE-16665
>                 URL: https://issues.apache.org/jira/browse/HBASE-16665
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Heng Chen
>         Attachments: HBASE-16665.patch
>
>




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