You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by el...@apache.org on 2019/06/13 20:05:07 UTC
[hbase] branch master updated: HBASE-22562 Remove dead code:
skipControl
This is an automated email from the ASF dual-hosted git repository.
elserj 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 35d7088 HBASE-22562 Remove dead code: skipControl
35d7088 is described below
commit 35d70881c39b1345e987aea2ea3a5e18f65bc7b1
Author: Josh Elser <el...@apache.org>
AuthorDate: Wed Jun 12 19:15:14 2019 -0400
HBASE-22562 Remove dead code: skipControl
Signed-off-by: Reid Chan <re...@apache.org>
---
.../throttle/PressureAwareCompactionThroughputController.java | 9 ---------
.../throttle/PressureAwareFlushThroughputController.java | 6 ------
.../regionserver/throttle/PressureAwareThroughputController.java | 8 --------
3 files changed, 23 deletions(-)
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 740152d..5052f30 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
@@ -146,13 +146,4 @@ public class PressureAwareCompactionThroughputController extends PressureAwareTh
+ throughputDesc(getMaxThroughput()) + ", activeCompactions=" + activeOperations.size()
+ "]";
}
-
- @Override
- protected boolean skipControl(long deltaSize, long controlSize) {
- if (deltaSize < controlSize) {
- return true;
- } else {
- return false;
- }
- }
}
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/throttle/PressureAwareFlushThroughputController.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/throttle/PressureAwareFlushThroughputController.java
index fac4e86..51e7b42 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/throttle/PressureAwareFlushThroughputController.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/throttle/PressureAwareFlushThroughputController.java
@@ -128,10 +128,4 @@ public class PressureAwareFlushThroughputController extends PressureAwareThrough
return "DefaultFlushController [maxThroughput=" + throughputDesc(getMaxThroughput())
+ ", activeFlushNumber=" + activeOperations.size() + "]";
}
-
- @Override
- protected boolean skipControl(long deltaSize, long controlSize) {
- // for flush, we control the flow no matter whether the flush size is small
- return false;
- }
}
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/throttle/PressureAwareThroughputController.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/throttle/PressureAwareThroughputController.java
index 27c25ea..306df0b 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/throttle/PressureAwareThroughputController.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/throttle/PressureAwareThroughputController.java
@@ -137,14 +137,6 @@ public abstract class PressureAwareThroughputController extends Configured imple
return sleepTime;
}
- /**
- * Check whether to skip control given delta size and control size
- * @param deltaSize Delta size since last control
- * @param controlSize Size limit to perform control
- * @return a boolean indicates whether to skip this control
- */
- protected abstract boolean skipControl(long deltaSize, long controlSize);
-
@Override
public void finish(String opName) {
ActiveOperation operation = activeOperations.remove(opName);