Apache
/
hbase
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

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 <stack@apache.org>
This commit is contained in:
Alex Leblang 2017-07-18 15:01:55 -07:00 committed by Michael Stack
parent d776a3caae
commit dbc8ea6486
1 changed files with 0 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/ServerCrashProcedure.java
View File

@ -139,7 +139,6 @@ implements ServerProcedureInterface {
switch (state) { switch (state) {
case SERVER_CRASH_START: case SERVER_CRASH_START:
LOG.info("Start " + this); LOG.info("Start " + this);
start(env);
// If carrying meta, process it first. Else, get list of regions on crashed server. // If carrying meta, process it first. Else, get list of regions on crashed server.
if (this.carryingMeta) { if (this.carryingMeta) {
setNextState(ServerCrashState.SERVER_CRASH_PROCESS_META); setNextState(ServerCrashState.SERVER_CRASH_PROCESS_META);
@ -206,13 +205,6 @@ implements ServerProcedureInterface {
return Flow.HAS_MORE_STATE; 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 * @param env
* @throws IOException * @throws IOException