You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This patch improves the logging path in YarnChild.reportError() when ClusterStorageCapacityExceededException is raised and fast fail is disabled. The task now emits a warning that explains why the job is not failing immediately and points users to the configuration that enables fast fail.
Change
hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapred/YarnChild.java: add a warning log message for the non-fast-fail capacity-exceeded path.
hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/test/java/org/apache/hadoop/mapred/TestYarnChild.java: extend the unit coverage to assert the warning path and preserve the existing fatal-error behavior checks.
Tests
JAVA_HOME=/opt/homebrew/opt/openjdk /opt/homebrew/bin/mvn -Dmaven.repo.local=/tmp/codex-m2 test -pl hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app -am -Dtest=TestYarnChild -DskipTests=false
Added a warning when cluster storage capacity is exceeded but fast fail is disabled, so users get guidance on enabling it. TestYarnChild passes locally with mvn -Dmaven.repo.local=/tmp/codex-m2 test -pl hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app -am -Dtest=TestYarnChild -DskipTests=false. Ready for CI.
CI is green on the current head, and this one still has not had a human review. @adoroszlai, when you have a moment, could you please take a look? Happy to make any follow-up changes.
CI is still green on the current head, and this one is waiting on a MapReduce/YARN review. @wangda@sunilg, when you have a moment, could you please take a look? Happy to make any follow-up changes.
CI is still green on the current head, and this one is still waiting on a MapReduce/YARN review. @wangda@sunilg, when you have a moment, could you please take a look? Happy to make any follow-up changes.
CI is still green on the current head, and this one is still waiting on a MapReduce/YARN review. @wangda@sunilg, when you have a moment, could you please take a look? Happy to make any follow-up changes.
CI is still green on the current head, and this one is still waiting on a MapReduce/YARN review. @wangda@sunilg, when you have a moment, could you please take a look? Happy to make any follow-up changes.
CI is still green on the current head, and this one is still waiting on a MapReduce/YARN review as of July 20, 2026. @wangda@sunilg, when you have a moment, could you please take a look? Happy to make any follow-up changes.
CI is still green on the current head as of July 26, 2026, and this one is still waiting on a MapReduce/YARN review. @wangda@sunilg, when you have a moment, could you please take a look? Happy to make any follow-up changes.
CI is still green on the current head as of July 29, 2026, and this one is still waiting on a MapReduce/YARN review. @wangda@sunilg, when you have a moment, could you please take a look? Happy to make any follow-up changes.
CI is still green on the current head as of August 4, 2026, and this one is still waiting on a MapReduce/YARN review. @wangda@sunilg, when you have a moment, could you please take a look? Happy to make any follow-up changes.
CI is still green on the current head as of August 7, 2026, and this one is still waiting on a MapReduce/YARN review. @wangda@sunilg, when you have a moment, could you please take a look? Happy to make any follow-up changes.
CI is still green on the current head as of August 10, 2026, and this one is still waiting on a MapReduce/YARN review. @wangda@sunilg, when you have a moment, could you please take a look? Happy to make any follow-up changes.
Hi @steveloughran@slfan1989@pan3793, MAPREDUCE-7486 has been open since March. Small YarnChild change plus a test, rebased on trunk with Yetus +1. Could someone review or merge if it looks good?
Rebased cleanly onto current trunk at 706d4dcb. git diff --check apache/trunk...origin/MAPREDUCE-7486-enhanced-storage-logging passes. This host currently has no Java runtime or Maven, so I could not rerun the focused Maven test here; fresh Build and Jenkins checks are queued on 3e7310de. Ready for CI.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This patch improves the logging path in
YarnChild.reportError()whenClusterStorageCapacityExceededExceptionis raised and fast fail is disabled. The task now emits a warning that explains why the job is not failing immediately and points users to the configuration that enables fast fail.Change
hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapred/YarnChild.java: add a warning log message for the non-fast-fail capacity-exceeded path.hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/test/java/org/apache/hadoop/mapred/TestYarnChild.java: extend the unit coverage to assert the warning path and preserve the existing fatal-error behavior checks.Tests
JAVA_HOME=/opt/homebrew/opt/openjdk /opt/homebrew/bin/mvn -Dmaven.repo.local=/tmp/codex-m2 test -pl hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app -am -Dtest=TestYarnChild -DskipTests=falseJIRA
Fixes MAPREDUCE-7486