HBASE-21777 Only log compaction pressure throughput when changed or traced

Signed-off-by: Yu Li <[email protected]>
Signed-off-by: Andrew Purtell <[email protected]>

Author: Tak Lon (Stephen) Wu

hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/throttle/PressureAwareCompactionThroughputController.java

@@ -99,11 +99,14 @@ private void tune(double compactionPressure) {
           maxThroughputLowerBound + (maxThroughputUpperBound - maxThroughputLowerBound)
               * compactionPressure;
     }
-    if (LOG.isTraceEnabled()) {
-      // TODO: FIX!!! Don't log unless some activity or a change in config. Making TRACE
-      // in the meantime.
-      LOG.trace("CompactionPressure is " + compactionPressure + ", tune throughput to "
-          + throughputDesc(maxThroughputToSet));
+    if (LOG.isDebugEnabled()) {
+      if (maxThroughputToSet != getMaxThroughput()) {
+        LOG.debug("CompactionPressure is " + compactionPressure + ", tune throughput to "
+            + throughputDesc(maxThroughputToSet));
+      } else if (LOG.isTraceEnabled()) {
+        LOG.trace("CompactionPressure is " + compactionPressure + ", keep throughput throttling to "
+            + throughputDesc(maxThroughputToSet));
+      }
     }
     this.setMaxThroughput(maxThroughputToSet);
   }