-1 tests included. The patch doesn't appear to include any new or modified tests.
The patch is adding a debug log message which is hard to write a unit test for. Our QE team has manually verified the patch by running tests on a cluster with old memory configuration and verifying that the number of WARN messages has come down drastically. Also, the scenario in
MAPREDUCE-833 has been tested and found fixed.
-1 core tests. The patch failed core unit tests.
org.apache.hadoop.mapred.TestRecoveryManager.testRestartCount's failure is independent of this patch. It happens with trunk and is being tracked in
-1 contrib tests. The patch failed contrib unit tests.
The streaming test failures have been happening for a while on hudson alone. They pass locally.
On this basis, I am going to commit the patch to trunk and an equivalent patch to branch 20.