You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by li...@apache.org on 2019/05/08 17:02:43 UTC

[hbase] branch branch-1 updated: HBASE-21777 Only log compaction pressure throughput when changed or traced (addendum)

This is an automated email from the ASF dual-hosted git repository.

liyu pushed a commit to branch branch-1
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-1 by this push:
     new e4ab715  HBASE-21777 Only log compaction pressure throughput when changed or traced (addendum)
e4ab715 is described below

commit e4ab71590142c8481fbedf15450504a5f698cbab
Author: Yu Li <li...@apache.org>
AuthorDate: Thu May 9 00:59:10 2019 +0800

    HBASE-21777 Only log compaction pressure throughput when changed or traced (addendum)
    
    Fix the findbugs warning
---
 .../throttle/PressureAwareCompactionThroughputController.java           | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/throttle/PressureAwareCompactionThroughputController.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/throttle/PressureAwareCompactionThroughputController.java
index 7df07bb..2dc5817 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/throttle/PressureAwareCompactionThroughputController.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/throttle/PressureAwareCompactionThroughputController.java
@@ -100,7 +100,7 @@ public class PressureAwareCompactionThroughputController extends PressureAwareTh
               * compactionPressure;
     }
     if (LOG.isDebugEnabled()) {
-      if (maxThroughputToSet != getMaxThroughput()) {
+      if (Math.abs(maxThroughputToSet - getMaxThroughput()) < .0000001) {
         LOG.debug("CompactionPressure is " + compactionPressure + ", tune throughput to "
             + throughputDesc(maxThroughputToSet));
       } else if (LOG.isTraceEnabled()) {