diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/TestYarnClient.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/TestYarnClient.java index fbc876a..8b61513 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/TestYarnClient.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/TestYarnClient.java @@ -145,7 +145,7 @@ public MockYarnClient() { } @Override - public void start() { + protected void serviceStart() throws Exception { rmClient = mock(ClientRMProtocol.class); GetApplicationReportResponse mockResponse = mock(GetApplicationReportResponse.class); @@ -162,7 +162,7 @@ public void start() { } @Override - public void stop() { + protected void serviceStop() throws Exception { } public void setYarnApplicationState(YarnApplicationState state) { diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/service/AbstractService.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/service/AbstractService.java index fc35f1c..ebaded6 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/service/AbstractService.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/service/AbstractService.java @@ -143,7 +143,7 @@ protected void setConfig(Configuration conf) { * the state change not permitted, or something else went wrong */ @Override - public void init(Configuration conf) { + public final void init(Configuration conf) { if (conf == null) { throw new ServiceStateException("Cannot initialize service " + getName() + ": null configuration"); @@ -176,7 +176,7 @@ public void init(Configuration conf) { * this action */ @Override - public void start() { + public final void start() { if (isInState(STATE.STARTED)) { return; } @@ -206,7 +206,7 @@ public void start() { * {@inheritDoc} */ @Override - public void stop() { + public final void stop() { if (isInState(STATE.STOPPED)) { return; }