You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by te...@apache.org on 2017/11/09 16:32:35 UTC

hbase git commit: HBASE-19199 RatioBasedCompactionPolicy#shouldPerformMajorCompaction() always return true when only one file needs to compact

Repository: hbase
Updated Branches:
  refs/heads/master 0bdfd4f44 -> ec59b0008


HBASE-19199 RatioBasedCompactionPolicy#shouldPerformMajorCompaction() always return true when only one file needs to compact

Signed-off-by: tedyu <yu...@gmail.com>


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/ec59b000
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/ec59b000
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/ec59b000

Branch: refs/heads/master
Commit: ec59b0008ccb3ff86496372285c17ddc6f916011
Parents: 0bdfd4f
Author: Guangxu Cheng <gu...@gmail.com>
Authored: Tue Nov 7 18:18:57 2017 +0800
Committer: tedyu <yu...@gmail.com>
Committed: Thu Nov 9 08:32:24 2017 -0800

----------------------------------------------------------------------
 .../hbase/regionserver/compactions/RatioBasedCompactionPolicy.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/ec59b000/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/compactions/RatioBasedCompactionPolicy.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/compactions/RatioBasedCompactionPolicy.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/compactions/RatioBasedCompactionPolicy.java
index 98f87e6..dba0473 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/compactions/RatioBasedCompactionPolicy.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/compactions/RatioBasedCompactionPolicy.java
@@ -106,8 +106,8 @@ public class RatioBasedCompactionPolicy extends SortedCompactionPolicy {
       } else {
         LOG.debug("Major compaction triggered on store " + regionInfo
           + "; time since last major compaction " + (now - lowTimestamp) + "ms");
+        result = true;
       }
-      result = true;
     }
     return result;
   }