From dbc8ea64862935c27414beaae73f12c25b56431c Mon Sep 17 00:00:00 2001 From: Alex Leblang Date: Tue, 18 Jul 2017 15:01:55 -0700 Subject: [PATCH] HBASE-18406 Remove NO-OP Method This patch removes start(MasterProcedureEnv) from ServerCrashProcedure.java which was a misnomer as a no-op. It did not start anything. Change-Id: I4e91864ace912e137471bfce03516746c4aff83e Signed-off-by: Michael Stack --- .../hbase/master/procedure/ServerCrashProcedure.java | 8 -------- 1 file changed, 8 deletions(-) diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/ServerCrashProcedure.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/ServerCrashProcedure.java index db4ac327bb0..4fb8c07f885 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/ServerCrashProcedure.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/ServerCrashProcedure.java @@ -139,7 +139,6 @@ implements ServerProcedureInterface { switch (state) { case SERVER_CRASH_START: LOG.info("Start " + this); - start(env); // If carrying meta, process it first. Else, get list of regions on crashed server. if (this.carryingMeta) { setNextState(ServerCrashState.SERVER_CRASH_PROCESS_META); @@ -206,13 +205,6 @@ implements ServerProcedureInterface { return Flow.HAS_MORE_STATE; } - /** - * Start processing of crashed server. In here we'll just set configs. and return. - * @param env - * @throws IOException - */ - private void start(final MasterProcedureEnv env) throws IOException {} - /** * @param env * @throws IOException