You are viewing a plain text version of this content. The canonical link for it is here.
Posted to yarn-dev@hadoop.apache.org by "Ming Ma (JIRA)" <ji...@apache.org> on 2014/02/22 01:55:19 UTC

[jira] [Created] (YARN-1750) TestNodeStatusUpdater#testNMRegistration incorrect

Ming Ma created YARN-1750:
-----------------------------

             Summary: TestNodeStatusUpdater#testNMRegistration incorrect
                 Key: YARN-1750
                 URL: https://issues.apache.org/jira/browse/YARN-1750
             Project: Hadoop YARN
          Issue Type: Bug
          Components: nodemanager
            Reporter: Ming Ma


This test case passes. However, the test output log has

java.lang.AssertionError: Number of applications should only be one! expected:<1> but was:<2>
        at org.junit.Assert.fail(Assert.java:93)
        at org.junit.Assert.failNotEquals(Assert.java:647)
        at org.junit.Assert.assertEquals(Assert.java:128)
        at org.junit.Assert.assertEquals(Assert.java:472)
        at org.apache.hadoop.yarn.server.nodemanager.TestNodeStatusUpdater$MyResourceTracker.nodeHeartbeat(TestNodeStatusUpdater.java:267)
        at org.apache.hadoop.yarn.server.nodemanager.NodeStatusUpdaterImpl$1.run(NodeStatusUpdaterImpl.java:469)
        at java.lang.Thread.run(Thread.java:695)

TestNodeStatusUpdater.java has invalid asserts.

      } else if (heartBeatID == 3) {
        // Checks on the RM end
        Assert.assertEquals("Number of applications should only be one!", 1,
            appToContainers.size());
        Assert.assertEquals("Number of container for the app should be two!",
            2, appToContainers.get(appId2).size());


We should fix the assert and add more check to the test.






--
This message was sent by Atlassian JIRA
(v6.1.5#6160)