HBASE-27807 PressureAwareCompactionThroughputController#tune log the opposite of the actual scenario (#5198)

haosenchen · Apache9 · commit 0e224b1ef6d5 · 2023-04-23T23:34:25.000+08:00
Signed-off-by: Duo Zhang <zhangduo@apache.org> (cherry picked from commit 95f83cb)
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
@@ -99,7 +99,7 @@ private void tune(double compactionPressure) {
         + (maxThroughputUpperBound - maxThroughputLowerBound) * compactionPressure;
     }
     if (LOG.isDebugEnabled()) {
-      if (Math.abs(maxThroughputToSet - getMaxThroughput()) < .0000001) {
+      if (Math.abs(maxThroughputToSet - getMaxThroughput()) >= .0000001) {
         LOG.debug("CompactionPressure is " + compactionPressure + ", tune throughput to "
           + throughputDesc(maxThroughputToSet));
       } else if (LOG.isTraceEnabled()) {

Original file line number	Diff line number	Diff line change
`@@ -99,7 +99,7 @@ private void tune(double compactionPressure) {`
`99`	`99`	`+ (maxThroughputUpperBound - maxThroughputLowerBound) * compactionPressure;`
`100`	`100`	`}`
`101`	`101`	`if (LOG.isDebugEnabled()) {`
`102`		`- if (Math.abs(maxThroughputToSet - getMaxThroughput()) < .0000001) {`
	`102`	`+ if (Math.abs(maxThroughputToSet - getMaxThroughput()) >= .0000001) {`
`103`	`103`	`LOG.debug("CompactionPressure is " + compactionPressure + ", tune throughput to "`
`104`	`104`	`+ throughputDesc(maxThroughputToSet));`
`105`	`105`	`} else if (LOG.isTraceEnabled()) {`