diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java index c21844a0548..62258978178 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java @@ -3408,7 +3408,7 @@ public class AssignmentManager extends ZooKeeperListener { return true; } - void invokeAssign(HRegionInfo regionInfo) { + public void invokeAssign(HRegionInfo regionInfo) { invokeAssign(regionInfo, true); } @@ -3421,12 +3421,6 @@ public class AssignmentManager extends ZooKeeperListener { new AssignCallable(this, regionInfo, true)), sleepMillis, TimeUnit.MILLISECONDS); } - public void invokeAssignLaterOnFailure(HRegionInfo regionInfo) { - long sleepTime = backoffPolicy.getBackoffTime(retryConfig, - failedOpenTracker.get(regionInfo.getEncodedName()).get()); - invokeAssignLater(regionInfo, sleepTime); - } - void invokeUnAssign(HRegionInfo regionInfo) { threadPoolExecutorService.submit(new UnAssignCallable(this, regionInfo)); } @@ -4498,8 +4492,10 @@ public class AssignmentManager extends ZooKeeperListener { } private class DelayedAssignCallable implements Runnable { - Callable callable; - public DelayedAssignCallable(Callable callable) { + + private final Callable callable; + + public DelayedAssignCallable(Callable callable) { this.callable = callable; } diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/handler/ClosedRegionHandler.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/handler/ClosedRegionHandler.java index 04106d821b6..448d483e861 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/handler/ClosedRegionHandler.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/handler/ClosedRegionHandler.java @@ -103,6 +103,6 @@ public class ClosedRegionHandler extends EventHandler implements TotesHRegionInf regionInfo, RegionState.State.CLOSED); // This below has to do w/ online enable/disable of a table assignmentManager.removeClosedRegion(regionInfo); - assignmentManager.invokeAssignLaterOnFailure(regionInfo); + assignmentManager.invokeAssign(regionInfo); } }