Lars George created HBASE-6472:
----------------------------------

             Summary: Bulk load files can cause minor compactions with single 
files
                 Key: HBASE-6472
                 URL: https://issues.apache.org/jira/browse/HBASE-6472
             Project: HBase
          Issue Type: Bug
          Components: regionserver
    Affects Versions: 0.94.0, 0.92.1, 0.90.6
            Reporter: Lars George


This is visible in the logs:

{noformat}
2012-07-17 00:00:29,081 INFO org.apache.hadoop.hbase.regionserver.Store: 
Started compaction of 1 file(s) in cf=m into 
hdfs://nn:9000/hbase/t1/27889b3c9257e8dff4cddae5eedfd064/.tmp, 
seqid=1918186410, totalSize=124.7m 
2012-07-17 00:00:29,081 DEBUG org.apache.hadoop.hbase.regionserver.Store: 
Compacting 
hdfs://nn:9000/hbase/t1/27889b3c9257e8dff4cddae5eedfd064/m/5478928132066935241, 
keycount=1435991, bloomtype=NONE, size=124.7m 
2012-07-17 00:00:34,538 INFO org.apache.hadoop.hbase.regionserver.Store: 
Completed major compaction of 1 file(s), new 
file=hdfs://nn:9000/hbase/t1/27889b3c9257e8dff4cddae5eedfd064/m/7554093203376406441,
 size=124.7m; total size for store is 124.7m
{noformat}

I assume the reason is that the Store.java check is reversed:

{code}
      // skip selection algorithm if we don't have enough files
      if (compactSelection.getFilesToCompact().size() < this.minFilesToCompact) 
{
        if(LOG.isDebugEnabled()) {
          LOG.debug("Not compacting files because we only have " +
            compactSelection.getFilesToCompact().size() +
            " files ready for compaction.  Need " + this.minFilesToCompact + " 
to initiate.");
        }
        compactSelection.emptyFileList();
        return compactSelection;
      }

      // remove bulk import files that request to be excluded from minors
      compactSelection.getFilesToCompact().removeAll(Collections2.filter(
          compactSelection.getFilesToCompact(),
          new Predicate<StoreFile>() {
            public boolean apply(StoreFile input) {
              return input.excludeFromMinorCompaction();
            }
          }));
{code}

The bulk files should be removed first, and +then+ it should check if there are 
enough files.

There are other places doing the same things, i.e. removing the bulk files and 
check what needs to be done, so this needs some extra care to get it all right.

--
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

        

Reply via email to