Repository: hbase
Updated Branches:
  refs/heads/branch-1.1 ba217ac07 -> 93fce057a


HBASE-13528 A bug on selecting compaction pool (Shuaifeng Zhou)


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

Branch: refs/heads/branch-1.1
Commit: 93fce057a306478319d8a0af001b06be273b36d0
Parents: ba217ac
Author: tedyu <[email protected]>
Authored: Fri Apr 24 02:20:44 2015 -0700
Committer: tedyu <[email protected]>
Committed: Fri Apr 24 02:20:44 2015 -0700

----------------------------------------------------------------------
 .../org/apache/hadoop/hbase/regionserver/CompactSplitThread.java  | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/93fce057/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/CompactSplitThread.java
----------------------------------------------------------------------
diff --git 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/CompactSplitThread.java
 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/CompactSplitThread.java
index 3e3e398..4e30897 100644
--- 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/CompactSplitThread.java
+++ 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/CompactSplitThread.java
@@ -346,8 +346,7 @@ public class CompactSplitThread implements 
CompactionRequestor, PropagatingConfi
 
     // We assume that most compactions are small. So, put system compactions 
into small
     // pool; we will do selection there, and move to large pool if necessary.
-    long size = selectNow ? compaction.getRequest().getSize() : 0;
-    ThreadPoolExecutor pool = (!selectNow && s.throttleCompaction(size))
+    ThreadPoolExecutor pool = (selectNow && 
s.throttleCompaction(compaction.getRequest().getSize()))
       ? longCompactions : shortCompactions;
     pool.execute(new CompactionRunner(s, r, compaction, pool));
     if (LOG.isDebugEnabled()) {

Reply via email to