diff --git a/hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/RemoteProcedureDispatcher.java b/hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/RemoteProcedureDispatcher.java index 9c1f5ca8a46..73f6d473c09 100644 --- a/hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/RemoteProcedureDispatcher.java +++ b/hbase-procedure/src/main/java/org/apache/hadoop/hbase/procedure2/RemoteProcedureDispatcher.java @@ -237,7 +237,7 @@ public abstract class RemoteProcedureDispatcher } @Override - public synchronized void remoteCallFailed(MasterProcedureEnv env, ServerName remote, + public synchronized boolean remoteCallFailed(MasterProcedureEnv env, ServerName remote, IOException exception) { complete(env, exception); + return true; } @Override diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/master/snapshot/TestAssignProcedure.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/master/snapshot/TestAssignProcedure.java index 8836c9f75e5..5067fc361c1 100644 --- a/hbase-server/src/test/java/org/apache/hadoop/hbase/master/snapshot/TestAssignProcedure.java +++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/master/snapshot/TestAssignProcedure.java @@ -108,11 +108,12 @@ public class TestAssignProcedure { } @Override - public void remoteCallFailed(final MasterProcedureEnv env, + public boolean remoteCallFailed(final MasterProcedureEnv env, final ServerName serverName, final IOException exception) { // Just skip this remoteCallFailed. Its too hard to mock. Assert it is called though. // Happens after the code we are testing has been called. this.remoteCallFailedWasCalled.set(true); + return true; } };