YARN-7357. Several methods in TestZKRMStateStore.TestZKRMStateStoreTester.TestZKRMStateStoreInternal should have @Override annotations (Contributed by Sen Zhao via Daniel Templeton)
Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/9e77dc2b Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/9e77dc2b Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/9e77dc2b Branch: refs/heads/HDFS-9806 Commit: 9e77dc2bd149a38a99c0e2668ee27c91f99cf8a3 Parents: 96be795 Author: Daniel Templeton <[email protected]> Authored: Mon Oct 23 13:49:30 2017 -0700 Committer: Daniel Templeton <[email protected]> Committed: Mon Oct 23 13:51:19 2017 -0700 ---------------------------------------------------------------------- .../server/resourcemanager/recovery/TestZKRMStateStore.java | 5 +++++ 1 file changed, 5 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hadoop/blob/9e77dc2b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/recovery/TestZKRMStateStore.java ---------------------------------------------------------------------- diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/recovery/TestZKRMStateStore.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/recovery/TestZKRMStateStore.java index e5cf95d..f94bbd3 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/recovery/TestZKRMStateStore.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/recovery/TestZKRMStateStore.java @@ -144,6 +144,7 @@ public class TestZKRMStateStore extends RMStateStoreTestBase { return znodeWorkingPath + "/" + ROOT_ZNODE_NAME + "/" + VERSION_NODE; } + @Override public Version getCurrentVersion() { return CURRENT_VERSION_INFO; } @@ -193,6 +194,7 @@ public class TestZKRMStateStore extends RMStateStoreTestBase { return createStore(conf); } + @Override public RMStateStore getRMStateStore() throws Exception { YarnConfiguration conf = new YarnConfiguration(); return createStore(conf); @@ -216,6 +218,7 @@ public class TestZKRMStateStore extends RMStateStoreTestBase { return store.getCurrentVersion(); } + @Override public boolean appExists(RMApp app) throws Exception { String appIdPath = app.getApplicationId().toString(); int split = @@ -225,6 +228,7 @@ public class TestZKRMStateStore extends RMStateStoreTestBase { .forPath(store.getAppNode(appIdPath, split)); } + @Override public boolean attemptExists(RMAppAttempt attempt) throws Exception { ApplicationAttemptId attemptId = attempt.getAppAttemptId(); return null != curatorFramework.checkExists() @@ -274,6 +278,7 @@ public class TestZKRMStateStore extends RMStateStoreTestBase { static class TestAppRejDispatcher extends TestDispatcher { private boolean appsavefailedEvnt; + @Override public void handle(Event event) { if (event instanceof RMAppEvent && event.getType() .equals(RMAppEventType.APP_SAVE_FAILED)) { --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
