hadoop-yarn-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ming Ma (JIRA)" <j...@apache.org>
Subject [jira] [Created] (YARN-1750) TestNodeStatusUpdater#testNMRegistration incorrect
Date Sat, 22 Feb 2014 00:55:19 GMT
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)

Mime
View raw message