[ https://issues.apache.org/jira/browse/YARN-2148?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14029250#comment-14029250 ]
Hudson commented on YARN-2148: ------------------------------ FAILURE: Integrated in Hadoop-Mapreduce-trunk #1799 (See [https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1799/]) YARN-2148. TestNMClient failed due more exit code values added and passed to AM (Wangda Tan via bikas) (bikas: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1602043) * /hadoop/common/trunk/hadoop-yarn-project/CHANGES.txt * /hadoop/common/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/test/java/org/apache/hadoop/yarn/client/api/impl/TestNMClient.java > TestNMClient failed due more exit code values added and passed to AM > -------------------------------------------------------------------- > > Key: YARN-2148 > URL: https://issues.apache.org/jira/browse/YARN-2148 > Project: Hadoop YARN > Issue Type: Bug > Components: client > Affects Versions: 3.0.0, 2.5.0 > Reporter: Wangda Tan > Assignee: Wangda Tan > Fix For: 2.5.0 > > Attachments: YARN-2148.patch > > > Currently, TestNMClient will be failed in trunk, see > https://builds.apache.org/job/PreCommit-YARN-Build/3959/testReport/junit/org.apache.hadoop.yarn.client.api.impl/TestNMClient/testNMClient/ > {code} > java.lang.AssertionError: null > at org.junit.Assert.fail(Assert.java:86) > at org.junit.Assert.assertTrue(Assert.java:41) > at org.junit.Assert.assertTrue(Assert.java:52) > at > org.apache.hadoop.yarn.client.api.impl.TestNMClient.testGetContainerStatus(TestNMClient.java:385) > at > org.apache.hadoop.yarn.client.api.impl.TestNMClient.testContainerManagement(TestNMClient.java:347) > at > org.apache.hadoop.yarn.client.api.impl.TestNMClient.testNMClient(TestNMClient.java:226) > {code} > Test cases in TestNMClient uses following code to verify exit code of > COMPLETED containers > {code} > testGetContainerStatus(container, i, ContainerState.COMPLETE, > "Container killed by the ApplicationMaster.", Arrays.asList( > new Integer[] {137, 143, 0})); > {code} > But YARN-2091 added logic to make exit code reflecting the actual status, so > exit code of the "killed by ApplicationMaster" will be -105, > {code} > if (container.hasDefaultExitCode()) { > container.exitCode = exitEvent.getExitCode(); > } > {code} > We should update test case as well. -- This message was sent by Atlassian JIRA (v6.2#6252)