[
https://issues.apache.org/jira/browse/HBASE-16444?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15432509#comment-15432509
]
Hudson commented on HBASE-16444:
--------------------------------
FAILURE: Integrated in Jenkins build HBase-Trunk_matrix #1466 (See
[https://builds.apache.org/job/HBase-Trunk_matrix/1466/])
HBASE-16444 CellUtil#estimatedSerializedSizeOfKey() should consider
(ramkrishna: rev 2aae923c32528667644c57085a0180bede1ad262)
* (edit) hbase-common/src/main/java/org/apache/hadoop/hbase/CellUtil.java
HBASE-16444 - Remove white space (ramkrishna: rev
2f7b9b5423ea7a5783197f049e5c15f4d544abeb)
* (edit) hbase-common/src/main/java/org/apache/hadoop/hbase/CellUtil.java
> CellUtil#estimatedSerializedSizeOfKey() should consider
> KEY_INFRASTRUCTURE_SIZE
> -------------------------------------------------------------------------------
>
> Key: HBASE-16444
> URL: https://issues.apache.org/jira/browse/HBASE-16444
> Project: HBase
> Issue Type: Bug
> Reporter: ramkrishna.s.vasudevan
> Assignee: ramkrishna.s.vasudevan
> Priority: Minor
> Fix For: 2.0.0, 1.4.0
>
> Attachments: HBASE-16444.patch, HBASE-16444_1.patch,
> HBASE-16444_2.patch
>
>
> Currently CellUtil#getSumOfCellKeyElementLengths() considers
> {code}
> return cell.getRowLength() + cell.getFamilyLength() +
> cell.getQualifierLength() +
> KeyValue.TIMESTAMP_TYPE_SIZE;
> {code}
> It can consider the 2 byte ROWLEN and 1 byte FAMILY_LEN also because with the
> current way of things we are sure how our key is structured.
> But pls note that
> {code}
> // This will be a low estimate. Will do for now.
> return getSumOfCellKeyElementLengths(cell);
> {code}
> It says clearly it is going to be a low estimate. But in the write path there
> should be no harm in adding the complete KEY_INFRA_SIZE.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
