[
https://issues.apache.org/jira/browse/MAPREDUCE-3872?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Patrick Hunt updated MAPREDUCE-3872:
------------------------------------
Attachment: MAPREDUCE-3872.patch
This patch fixes three issues:
1) ContainerLauncherImpl had a race in updating "allNodes". This might result
in the pool size being incorrectly calculated.
2) test had an issues where waitForEvent could run forever
3) there's a race btw eventHandlingThread in ContainerLauncherImpl and the test
code. In some cases the event would be processed, and the pool size increased,
before all events had been processed (and allNodes updated with all new hosts).
I believe this is the result of the test failure I documented in the
description.
> event handling races in ContainerLauncherImpl and TestContainerLauncher
> -----------------------------------------------------------------------
>
> Key: MAPREDUCE-3872
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-3872
> Project: Hadoop Map/Reduce
> Issue Type: Bug
> Components: client
> Affects Versions: 0.23.1
> Reporter: Patrick Hunt
> Attachments: MAPREDUCE-3872.patch
>
>
> TestContainerLauncher is failing intermittently for me.
> {noformat}
> junit.framework.AssertionFailedError: Expected: <null> but was: Expected 22
> but found 21
> at junit.framework.Assert.fail(Assert.java:47)
> at junit.framework.Assert.assertTrue(Assert.java:20)
> at junit.framework.Assert.assertNull(Assert.java:233)
> at junit.framework.Assert.assertNull(Assert.java:226)
> at
> org.apache.hadoop.mapreduce.v2.app.launcher.TestContainerLauncher.testPoolSize(TestContainerLauncher.java:117)
> {noformat}
> Patch momentarily.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
