diff --git a/hadoop-yarn-project/CHANGES.txt b/hadoop-yarn-project/CHANGES.txt index 60ac9defb2c..c7a8ef1a9ff 100644 --- a/hadoop-yarn-project/CHANGES.txt +++ b/hadoop-yarn-project/CHANGES.txt @@ -119,6 +119,9 @@ Release 2.4.1 - UNRELEASED YARN-1931. Private API change in YARN-1824 in 2.4 broke compatibility with previous releases (Sandy Ryza via tgraves) + + YARN-1750. TestNodeStatusUpdater#testNMRegistration is incorrect in test + case. (Wangda Tan via junping_du) Release 2.4.0 - 2014-04-07 diff --git a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/TestNodeStatusUpdater.java b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/TestNodeStatusUpdater.java index 76c544029c3..0ff34a51725 100644 --- a/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/TestNodeStatusUpdater.java +++ b/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/test/java/org/apache/hadoop/yarn/server/nodemanager/TestNodeStatusUpdater.java @@ -261,10 +261,12 @@ public class TestNodeStatusUpdater { this.context.getContainers().put(secondContainerID, container); } else if (heartBeatID == 3) { // Checks on the RM end - Assert.assertEquals("Number of applications should only be one!", 1, + Assert.assertEquals("Number of applications should have two!", 2, appToContainers.size()); - Assert.assertEquals("Number of container for the app should be two!", - 2, appToContainers.get(appId2).size()); + Assert.assertEquals("Number of container for the app-1 should be only one!", + 1, appToContainers.get(appId1).size()); + Assert.assertEquals("Number of container for the app-2 should be only one!", + 1, appToContainers.get(appId2).size()); // Checks on the NM end ConcurrentMap activeContainers =