[ https://issues.apache.org/jira/browse/MAPREDUCE-4924?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13554925#comment-13554925 ]
Hudson commented on MAPREDUCE-4924: ----------------------------------- Integrated in Hadoop-Yarn-trunk #98 (See [https://builds.apache.org/job/Hadoop-Yarn-trunk/98/]) MAPREDUCE-4924. flakey test: org.apache.hadoop.mapred.TestClusterMRNotification.testMR. (rkanter via tucu) (Revision 1433753) Result = SUCCESS tucu : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1433753 Files : * /hadoop/common/trunk/hadoop-mapreduce-project/CHANGES.txt * /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-jobclient/src/test/java/org/apache/hadoop/mapred/NotificationTestCase.java > flakey test: org.apache.hadoop.mapred.TestClusterMRNotification.testMR > ---------------------------------------------------------------------- > > Key: MAPREDUCE-4924 > URL: https://issues.apache.org/jira/browse/MAPREDUCE-4924 > Project: Hadoop Map/Reduce > Issue Type: Bug > Components: mrv1 > Affects Versions: 2.0.2-alpha > Reporter: Robert Kanter > Assignee: Robert Kanter > Priority: Trivial > Fix For: 1.2.0, 2.0.3-alpha > > Attachments: MAPREDUCE-4924_branch-1.patch, > MAPREDUCE-4924_branch-1.patch, MAPREDUCE-4924.patch, MAPREDUCE-4924.patch > > > I occasionally get a failure like this on > {{org.apache.hadoop.mapred.TestClusterMRNotification.testMR}} > {code} > junit.framework.AssertionFailedError: expected:<6> but was:<4> > at junit.framework.Assert.fail(Assert.java:47) > at junit.framework.Assert.failNotEquals(Assert.java:283) > at junit.framework.Assert.assertEquals(Assert.java:64) > at junit.framework.Assert.assertEquals(Assert.java:195) > at junit.framework.Assert.assertEquals(Assert.java:201) > at > org.apache.hadoop.mapred.NotificationTestCase.testMR(NotificationTestCase.java:184) > ... > {code} > It looks like a race condition: > {code} > // run a job with FAILED status > System.out.println(UtilsForTests.runJobFail(this.createJobConf(), inDir, > outDir).getID()); > Thread.sleep(2000); > assertEquals(6, NotificationServlet.counter); > assertEquals(0, NotificationServlet.failureCounter); > {code} > Instead of sleeping for 2 seconds, we should keep checking the counter and > fail after a timeout. There's a couple of similar places in the test that > should be fixed too. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira