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:09 UTC

[hbase] branch master 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 master
in repository https://gitbox.apache.org/repos/asf/hbase.git


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

commit 12bf75bb2aac8d519f607c50d4831a6bd9799875
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 63aa646..740152d 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()) {