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

YARN-7357. Several methods in TestZKRMStateStore.TestZKRMStateStoreTester.TestZKRMStateStoreInternal should have @Override annotations 

(Contributed by Sen Zhao via Daniel Templeton)
This commit is contained in:
Daniel Templeton 2017-10-23 13:49:30 -07:00
parent 96be795656
commit 9e77dc2bd1
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/recovery/TestZKRMStateStore.java
View File

@ -144,6 +144,7 @@ public class TestZKRMStateStore extends RMStateStoreTestBase {
return znodeWorkingPath + "/" + ROOT_ZNODE_NAME + "/" + VERSION_NODE; return znodeWorkingPath + "/" + ROOT_ZNODE_NAME + "/" + VERSION_NODE;
} }
@Override
public Version getCurrentVersion() { public Version getCurrentVersion() {
return CURRENT_VERSION_INFO; return CURRENT_VERSION_INFO;
} }
@ -193,6 +194,7 @@ public class TestZKRMStateStore extends RMStateStoreTestBase {
return createStore(conf); return createStore(conf);
} }
@Override
public RMStateStore getRMStateStore() throws Exception { public RMStateStore getRMStateStore() throws Exception {
YarnConfiguration conf = new YarnConfiguration(); YarnConfiguration conf = new YarnConfiguration();
return createStore(conf); return createStore(conf);
@ -216,6 +218,7 @@ public class TestZKRMStateStore extends RMStateStoreTestBase {
return store.getCurrentVersion(); return store.getCurrentVersion();
} }
@Override
public boolean appExists(RMApp app) throws Exception { public boolean appExists(RMApp app) throws Exception {
String appIdPath = app.getApplicationId().toString(); String appIdPath = app.getApplicationId().toString();
int split = int split =
@ -225,6 +228,7 @@ public class TestZKRMStateStore extends RMStateStoreTestBase {
.forPath(store.getAppNode(appIdPath, split)); .forPath(store.getAppNode(appIdPath, split));
} }
@Override
public boolean attemptExists(RMAppAttempt attempt) throws Exception { public boolean attemptExists(RMAppAttempt attempt) throws Exception {
ApplicationAttemptId attemptId = attempt.getAppAttemptId(); ApplicationAttemptId attemptId = attempt.getAppAttemptId();
return null != curatorFramework.checkExists() return null != curatorFramework.checkExists()
@ -274,6 +278,7 @@ public class TestZKRMStateStore extends RMStateStoreTestBase {
static class TestAppRejDispatcher extends TestDispatcher { static class TestAppRejDispatcher extends TestDispatcher {
private boolean appsavefailedEvnt; private boolean appsavefailedEvnt;
@Override
public void handle(Event event) { public void handle(Event event) {
if (event instanceof RMAppEvent && event.getType() if (event instanceof RMAppEvent && event.getType()
.equals(RMAppEventType.APP_SAVE_FAILED)) { .equals(RMAppEventType.APP_SAVE_FAILED)) {