-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.hbase.org/r/372/
-----------------------------------------------------------
(Updated 2010-07-24 05:10:08.798554)
Review request for hbase.
Changes
-------
Since currently there are only two log cleaners (TimeToLiveLogCleaner and
ReplicationLogCleaner) in the chain, there would be 3 scenarios, that is log
file doesn't pass the first log cleaner; log file passes the first log cleaner
but is rejected by the second; log file passes both log cleaners and is then
deleted. I think these 3 cases are all covered by the unit test. I've added
some comments to the unit test to explain these 3 cases.
Summary
-------
HBASE-2792: Create a better way to chain log cleaners
This addresses bug HBASE-2792.
http://issues.apache.org/jira/browse/HBASE-2792
Diffs (updated)
-----
src/main/java/org/apache/hadoop/hbase/master/LogCleanerDelegate.java 3ca3611
src/main/java/org/apache/hadoop/hbase/master/LogsCleaner.java 37b2c3c
src/main/java/org/apache/hadoop/hbase/master/ServerManager.java 9fb1cce
src/main/java/org/apache/hadoop/hbase/replication/master/ReplicationLogCleaner.java
eb859aa
src/main/resources/hbase-default.xml e3a9669
src/test/java/org/apache/hadoop/hbase/master/TestLogsCleaner.java a92e0da
Diff: http://review.hbase.org/r/372/diff
Testing
-------
Unit test TestOldLogsCleaner passed.
Thanks,
Chongxin
