Skip to content

HADOOP-19988: Register /mapreduce/app in MR AM tracking URL - #8727

Open
riya2305 wants to merge 1 commit into
apache:trunkfrom
riya2305:MAPREDUCE-fix-am-tracking-url
Open

riya2305 wants to merge 1 commit into
apache:trunkfrom
riya2305:MAPREDUCE-fix-am-tracking-url

Conversation

@riya2305

@riya2305 riya2305 commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Description of PR

On the YARN UI clicking on the application tracking URL results in 404; Appending /mapreduce/app at the end of URL will render to the correct page.

How was this patch tested?

tested patch manually

For code changes:

  • Does the title of this PR start with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
  • Object storage: Have the integration tests been executed and the endpoint
    declared according to the connector-specific documentation? Note: Automated CI
    testing doesn't cover all cases so manual testing with cloud storage is still
    required.
  • If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
  • If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

@hadoop-yetus

Copy link
Copy Markdown

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 55s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 1s codespell was not available.
+0 🆗 detsecrets 0m 1s detect-secrets was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
+1 💚 test4tests 0m 0s The patch appears to include 1 new or modified test files.
_ trunk Compile Tests _
+0 🆗 mvndep 2m 5s Maven dependency ordering for branch
+1 💚 mvninstall 51m 19s trunk passed
+1 💚 compile 17m 50s trunk passed with JDK Ubuntu-21.0.12+8-1-24.04-Ubuntu
+1 💚 compile 18m 11s trunk passed with JDK Ubuntu-17.0.20+8-1-24.04-Ubuntu
+1 💚 checkstyle 6m 1s trunk passed
+1 💚 mvnsite 2m 54s trunk passed
+1 💚 javadoc 2m 34s trunk passed with JDK Ubuntu-21.0.12+8-1-24.04-Ubuntu
+1 💚 javadoc 2m 34s trunk passed with JDK Ubuntu-17.0.20+8-1-24.04-Ubuntu
+1 💚 spotbugs 4m 15s trunk passed
+1 💚 shadedclient 35m 8s branch has no errors when building and testing our client artifacts.
-0 ⚠️ patch 35m 48s Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 29s Maven dependency ordering for patch
+1 💚 mvninstall 1m 22s the patch passed
+1 💚 compile 18m 31s the patch passed with JDK Ubuntu-21.0.12+8-1-24.04-Ubuntu
+1 💚 javac 18m 31s the patch passed
+1 💚 compile 21m 1s the patch passed with JDK Ubuntu-17.0.20+8-1-24.04-Ubuntu
+1 💚 javac 21m 1s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 checkstyle 6m 3s the patch passed
+1 💚 mvnsite 2m 52s the patch passed
+1 💚 javadoc 2m 38s the patch passed with JDK Ubuntu-21.0.12+8-1-24.04-Ubuntu
+1 💚 javadoc 2m 38s the patch passed with JDK Ubuntu-17.0.20+8-1-24.04-Ubuntu
+1 💚 spotbugs 4m 45s the patch passed
+1 💚 shadedclient 35m 19s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 unit 1m 57s hadoop-yarn-server-web-proxy in the patch passed.
+1 💚 unit 1m 31s hadoop-mapreduce-client-common in the patch passed.
+1 💚 unit 8m 58s hadoop-mapreduce-client-app in the patch passed.
+1 💚 asflicense 1m 14s The patch does not generate ASF License warnings.
261m 19s
Subsystem Report/Notes
Docker ClientAPI=1.56 ServerAPI=1.56 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8727/1/artifact/out/Dockerfile
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets
uname Linux 9c18c410cdfd 5.15.0-185-generic #195-Ubuntu SMP Fri Jun 19 17:11:50 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / 7609530
Default Java Ubuntu-17.0.20+8-1-24.04-Ubuntu
Multi-JDK versions /usr/lib/jvm/java-21-openjdk-amd64:Ubuntu-21.0.12+8-1-24.04-Ubuntu /usr/lib/jvm/java-17-openjdk-amd64:Ubuntu-17.0.20+8-1-24.04-Ubuntu
Test Results https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8727/1/testReport/
Max. process+thread count 581 (vs. ulimit of 10000)
modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app U: .
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8727/1/console
versions git=2.43.0 maven=3.9.15 spotbugs=4.9.7
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

@riya2305
riya2305 force-pushed the MAPREDUCE-fix-am-tracking-url branch from dfea637 to 6b4f941 Compare September 10, 2026 05:57

@Hean-Chhinling Hean-Chhinling left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the patch @riya2305!
LGTM!

@hadoop-yetus

Copy link
Copy Markdown

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 7m 2s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 0s codespell was not available.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
+1 💚 test4tests 0m 0s The patch appears to include 1 new or modified test files.
_ trunk Compile Tests _
+0 🆗 mvndep 2m 9s Maven dependency ordering for branch
+1 💚 mvninstall 26m 16s trunk passed
+1 💚 compile 8m 54s trunk passed with JDK Ubuntu-21.0.12+8-1-24.04-Ubuntu
+1 💚 compile 8m 55s trunk passed with JDK Ubuntu-17.0.20+8-1-24.04-Ubuntu
+1 💚 checkstyle 3m 4s trunk passed
+1 💚 mvnsite 1m 16s trunk passed
+1 💚 javadoc 1m 14s trunk passed with JDK Ubuntu-21.0.12+8-1-24.04-Ubuntu
+1 💚 javadoc 1m 11s trunk passed with JDK Ubuntu-17.0.20+8-1-24.04-Ubuntu
+1 💚 spotbugs 1m 50s trunk passed
+1 💚 shadedclient 16m 10s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 17s Maven dependency ordering for patch
+1 💚 mvninstall 0m 37s the patch passed
+1 💚 compile 8m 13s the patch passed with JDK Ubuntu-21.0.12+8-1-24.04-Ubuntu
+1 💚 javac 8m 13s the patch passed
+1 💚 compile 8m 54s the patch passed with JDK Ubuntu-17.0.20+8-1-24.04-Ubuntu
+1 💚 javac 8m 54s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 checkstyle 2m 56s the patch passed
+1 💚 mvnsite 1m 21s the patch passed
+1 💚 javadoc 1m 11s the patch passed with JDK Ubuntu-21.0.12+8-1-24.04-Ubuntu
+1 💚 javadoc 1m 9s the patch passed with JDK Ubuntu-17.0.20+8-1-24.04-Ubuntu
+1 💚 spotbugs 1m 53s the patch passed
+1 💚 shadedclient 16m 10s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 unit 1m 22s hadoop-yarn-server-web-proxy in the patch passed.
+1 💚 unit 6m 41s hadoop-mapreduce-client-app in the patch passed.
+1 💚 asflicense 0m 48s The patch does not generate ASF License warnings.
135m 12s
Subsystem Report/Notes
Docker ClientAPI=1.56 ServerAPI=1.56 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8727/2/artifact/out/Dockerfile
GITHUB PR #8727
Optional Tests dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets
uname Linux 4bca877132c4 5.15.0-190-generic #200-Ubuntu SMP Fri Aug 7 15:06:04 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / 6b4f941
Default Java Ubuntu-17.0.20+8-1-24.04-Ubuntu
Multi-JDK versions /usr/lib/jvm/java-21-openjdk-amd64:Ubuntu-21.0.12+8-1-24.04-Ubuntu /usr/lib/jvm/java-17-openjdk-amd64:Ubuntu-17.0.20+8-1-24.04-Ubuntu
Test Results https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8727/2/testReport/
Max. process+thread count 752 (vs. ulimit of 10000)
modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-web-proxy hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app U: .
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8727/2/console
versions git=2.43.0 maven=3.9.15 spotbugs=4.9.7
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

@riya2305

Copy link
Copy Markdown
Contributor Author

@slfan1989 can you please review and merge this, thanks.

@Hean-Chhinling Hean-Chhinling left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After looking at it more closer, I could some flaws on this PR.

  1. Shouldn't the test be at TestAMWebApp.java instead of TestProxyUriUtils.java?
  2. I think the ticket shall be at MAPREDUCE project instead of HADOOP.
  3. I found a similar changes before here. Shall it be linked with each other? https://issues.apache.org/jira/browse/MAPREDUCE-7533
    And it would be nice to explain what was wrong with MAPREDUCE-7533 that why this PR is needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants