HBASE-812 Compaction needs little better skip algo

git-svn-id: https://svn.apache.org/repos/asf/hadoop/hbase/trunk@685030 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Michael Stack 2008-08-12 04:14:24 +00:00
parent 3d6b96aa48
commit a28a8219cb
2 changed files with 12 additions and 2 deletions

View File

@ -16,6 +16,7 @@ Release 0.3.0 - Unreleased
HBASE-801 When a table haven't disable, shell could response in a "user HBASE-801 When a table haven't disable, shell could response in a "user
friendly" way. friendly" way.
HBASE-816 TableMap should survive USE (Andrew Purtell via Stack) HBASE-816 TableMap should survive USE (Andrew Purtell via Stack)
HBASE-812 Compaction needs little better skip algo (Daniel Leffel via Stack)
NEW FEATURES NEW FEATURES
OPTIMIZATIONS OPTIMIZATIONS

View File

@ -784,10 +784,19 @@ public class HStore implements HConstants {
} }
filesToCompact = new ArrayList<HStoreFile>(filesToCompact.subList(point, filesToCompact = new ArrayList<HStoreFile>(filesToCompact.subList(point,
countOfFiles)); countOfFiles));
if (filesToCompact.size() <= 1) {
if (LOG.isDebugEnabled()) {
LOG.debug("Skipped compaction of 1 file; compaction size of " +
this.storeNameStr + ": " +
StringUtils.humanReadableInt(totalSize) + "; Skipped " + point +
" files, size: " + skipped);
}
return checkSplit();
}
if (LOG.isDebugEnabled()) { if (LOG.isDebugEnabled()) {
LOG.debug("Compaction size of " + this.storeNameStr + ": " + LOG.debug("Compaction size of " + this.storeNameStr + ": " +
StringUtils.humanReadableInt(totalSize) + ", skipped " + point + StringUtils.humanReadableInt(totalSize) + "; Skipped " + point +
", " + skipped); " files , size: " + skipped);
} }
} }