Repository: hbase
Updated Branches:
  refs/heads/branch-1 0b86c7522 -> 42ad84503


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/42ad8450
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/42ad8450
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/42ad8450

Branch: refs/heads/branch-1
Commit: 42ad845032de466ef01912c1b5a0d7386a86aee9
Parents: 0b86c75
Author: tedyu <[email protected]>
Authored: Fri Apr 24 02:19:31 2015 -0700
Committer: tedyu <[email protected]>
Committed: Fri Apr 24 02:19:31 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/42ad8450/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