[
https://issues.apache.org/jira/browse/HBASE-6597?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13470475#comment-13470475
]
Phabricator commented on HBASE-6597:
------------------------------------
tedyu has commented on the revision "[jira] [HBASE-6597] [89-fb] Incremental
data block encoding".
INLINE COMMENTS
src/main/java/org/apache/hadoop/hbase/io/encoding/BufferedDataBlockEncoder.java:38
Length of prefix is returned.
Name this method getCommonPrefixLength ?
src/main/java/org/apache/hadoop/hbase/io/encoding/CopyKeyDataBlockEncoder.java:93
Do we need to consider memstoreTS ?
src/main/java/org/apache/hadoop/hbase/io/encoding/DiffKeyDeltaEncoder.java:340
Check / assert that skipLastBytes is not negative ?
src/main/java/org/apache/hadoop/hbase/io/encoding/DiffKeyDeltaEncoder.java:422
prevKey stores the previous key, right ?
src/main/java/org/apache/hadoop/hbase/io/encoding/DiffKeyDeltaEncoder.java:497
Name this variable negativeDiffTimestamp ?
src/main/java/org/apache/hadoop/hbase/io/encoding/FastDiffDeltaEncoder.java:411
This class can be private, right ?
src/main/java/org/apache/hadoop/hbase/io/encoding/DiffKeyDeltaEncoder.java:420
This class can be private, right ?
REVISION DETAIL
https://reviews.facebook.net/D5895
To: Kannan, Karthik, Liyin, aaiyer, avf, JIRA, mbautin
Cc: tedyu
> Block Encoding Size Estimation
> ------------------------------
>
> Key: HBASE-6597
> URL: https://issues.apache.org/jira/browse/HBASE-6597
> Project: HBase
> Issue Type: Improvement
> Components: io
> Affects Versions: 0.89-fb
> Reporter: Brian Nixon
> Priority: Minor
> Attachments: D5895.1.patch
>
>
> Blocks boundaries as created by current writers are determined by the size of
> the unencoded data. However, blocks in memory are kept encoded. By using an
> estimate for the encoded size of the block, we can get greater consistency in
> size.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
