diff --git a/hadoop-mapreduce-project/CHANGES.txt b/hadoop-mapreduce-project/CHANGES.txt index 93b11a07264..52a4e7ef175 100644 --- a/hadoop-mapreduce-project/CHANGES.txt +++ b/hadoop-mapreduce-project/CHANGES.txt @@ -1500,6 +1500,9 @@ Release 0.23.0 - Unreleased MAPREDUCE-2794. [MR-279] Incorrect metrics value for AvailableGB per queue per user. (John George via mahadev) + MAPREDUCE-2783. Fixing RM web-UI to show no tracking-URL when AM + crashes. (Eric Payne via vinodkv) + Release 0.22.0 - Unreleased INCOMPATIBLE CHANGES diff --git a/hadoop-mapreduce-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/rmapp/attempt/RMAppAttemptImpl.java b/hadoop-mapreduce-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/rmapp/attempt/RMAppAttemptImpl.java index 81aae502015..0141fedc780 100644 --- a/hadoop-mapreduce-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/rmapp/attempt/RMAppAttemptImpl.java +++ b/hadoop-mapreduce-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/rmapp/attempt/RMAppAttemptImpl.java @@ -777,6 +777,15 @@ public class RMAppAttemptImpl implements RMAppAttempt { " due to: " + containerStatus.getDiagnostics() + "." + "Failing this attempt."); + /* + * In the case when the AM dies, the trackingUrl is left pointing to the AM's + * URL, which shows up in the scheduler UI as a broken link. Setting it here + * to empty string will prevent any link from being displayed. + * NOTE: don't set trackingUrl to 'null'. That will cause null-pointer exceptions + * in the generated proto code. + */ + appAttempt.trackingUrl = ""; + new FinalTransition(RMAppAttemptState.FAILED).transition( appAttempt, containerFinishedEvent); return RMAppAttemptState.FAILED;