[ https://issues.apache.org/jira/browse/HBASE-4608?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13181869#comment-13181869 ]
jirapos...@reviews.apache.org commented on HBASE-4608: ------------------------------------------------------ ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2740/#review4232 ----------------------------------------------------------- src/main/java/org/apache/hadoop/hbase/regionserver/wal/Compressor.java <https://reviews.apache.org/r/2740/#comment9566> Add javadoc please. src/main/java/org/apache/hadoop/hbase/regionserver/wal/Compressor.java <https://reviews.apache.org/r/2740/#comment9567> Please give this config parameter better name. How about 'hbase.regionserver.wal.compressed' ? src/main/java/org/apache/hadoop/hbase/regionserver/wal/Compressor.java <https://reviews.apache.org/r/2740/#comment9568> Would this be able to hold large number of HLog.Entry's in memory ? src/main/java/org/apache/hadoop/hbase/regionserver/wal/Compressor.java <https://reviews.apache.org/r/2740/#comment9569> Since short is signed, how do I know that the return value would be positive ? e.g. (short)0xFE00 == -512 src/main/java/org/apache/hadoop/hbase/regionserver/wal/LRUDictionary.java <https://reviews.apache.org/r/2740/#comment9570> I suggest naming this class Node. src/main/java/org/apache/hadoop/hbase/regionserver/wal/WALEdit.java <https://reviews.apache.org/r/2740/#comment9573> Is compressed a better name ? src/main/java/org/apache/hadoop/hbase/regionserver/wal/WALEdit.java <https://reviews.apache.org/r/2740/#comment9574> White space makes indentation look weird. src/test/java/org/apache/hadoop/hbase/regionserver/wal/TestLRUDictionary.java <https://reviews.apache.org/r/2740/#comment9575> Please add Apache license. src/test/java/org/apache/hadoop/hbase/regionserver/wal/TestLRUDictionary.java <https://reviews.apache.org/r/2740/#comment9576> Add short javadoc and test category, please. src/test/java/org/apache/hadoop/hbase/regionserver/wal/TestWALReplayCompressed.java <https://reviews.apache.org/r/2740/#comment9572> Please remove year. src/test/java/org/apache/hadoop/hbase/regionserver/wal/TestWALReplayCompressed.java <https://reviews.apache.org/r/2740/#comment9571> You will add the real test, right ? Also, missing test category. - Ted On 2012-01-07 03:13:33, Li Pi wrote: bq. bq. ----------------------------------------------------------- bq. This is an automatically generated e-mail. To reply, visit: bq. https://reviews.apache.org/r/2740/ bq. ----------------------------------------------------------- bq. bq. (Updated 2012-01-07 03:13:33) bq. bq. bq. Review request for hbase, Eli Collins and Todd Lipcon. bq. bq. bq. Summary bq. ------- bq. bq. HLog compression. Has unit tests and a command line tool for compressing/decompressing. bq. bq. bq. This addresses bug HBase-4608. bq. https://issues.apache.org/jira/browse/HBase-4608 bq. bq. bq. Diffs bq. ----- bq. bq. src/main/java/org/apache/hadoop/hbase/HConstants.java 5120a3c bq. src/main/java/org/apache/hadoop/hbase/regionserver/wal/CompressedKeyValue.java PRE-CREATION bq. src/main/java/org/apache/hadoop/hbase/regionserver/wal/CompressionContext.java PRE-CREATION bq. src/main/java/org/apache/hadoop/hbase/regionserver/wal/Compressor.java PRE-CREATION bq. src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLog.java 24407af bq. src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLogKey.java f067221 bq. src/main/java/org/apache/hadoop/hbase/regionserver/wal/LRUDictionary.java PRE-CREATION bq. src/main/java/org/apache/hadoop/hbase/regionserver/wal/SequenceFileLogReader.java d9cd6de bq. src/main/java/org/apache/hadoop/hbase/regionserver/wal/SequenceFileLogWriter.java cbef70f bq. src/main/java/org/apache/hadoop/hbase/regionserver/wal/WALDictionary.java PRE-CREATION bq. src/main/java/org/apache/hadoop/hbase/regionserver/wal/WALEdit.java e1117ef bq. src/test/java/org/apache/hadoop/hbase/regionserver/wal/TestLRUDictionary.java PRE-CREATION bq. src/test/java/org/apache/hadoop/hbase/regionserver/wal/TestWALReplay.java 59910bf bq. src/test/java/org/apache/hadoop/hbase/regionserver/wal/TestWALReplayCompressed.java PRE-CREATION bq. bq. Diff: https://reviews.apache.org/r/2740/diff bq. bq. bq. Testing bq. ------- bq. bq. bq. Thanks, bq. bq. Li bq. bq. > HLog Compression > ---------------- > > Key: HBASE-4608 > URL: https://issues.apache.org/jira/browse/HBASE-4608 > Project: HBase > Issue Type: New Feature > Reporter: Li Pi > Assignee: Li Pi > Attachments: 4608v1.txt, 4608v5.txt, 4608v6.txt, 4608v7.txt, > 4608v8fixed.txt > > > The current bottleneck to HBase write speed is replicating the WAL appends > across different datanodes. We can speed up this process by compressing the > HLog. Current plan involves using a dictionary to compress table name, region > id, cf name, and possibly other bits of repeated data. Also, HLog format may > be changed in other ways to produce a smaller HLog. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira