Skip to content

Conversation

@brumi1024
Copy link
Member

@brumi1024 brumi1024 commented Nov 11, 2025

Description of PR

Migrate the new YARN Scheduler UI, and ensure maven builds it when the -Pyarn-ui flag is present. For details and the design doc please see the jira YARN-11885.

How was this patch tested?

Ran the build command and checked the artifacts.

Screenshot 2025-11-11 at 22 00 21 Screenshot 2025-11-11 at 22 00 33 Screenshot 2025-11-11 at 22 00 48 Screenshot 2025-11-11 at 22 01 02 Screenshot 2025-11-11 at 22 01 17 Screenshot 2025-11-11 at 22 01 29 Screenshot 2025-11-11 at 22 01 37 Screenshot 2025-11-11 at 22 01 48

For code changes:

  • Does the title or this PR starts 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?
  • 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?

@brumi1024 brumi1024 changed the title YARN-11886: Introduce Capacity Scheduler UI. YARN-11886. Introduce Capacity Scheduler UI. Nov 11, 2025
@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 0s Docker mode activated.
-1 ❌ patch 0m 21s #8076 does not apply to trunk. Rebase required? Wrong Branch? See https://cwiki.apache.org/confluence/display/HADOOP/How+To+Contribute for help.
Subsystem Report/Notes
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8076/1/console
versions git=2.34.1
Powered by Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 0s Docker mode activated.
-1 ❌ patch 0m 22s #8076 does not apply to trunk. Rebase required? Wrong Branch? See https://cwiki.apache.org/confluence/display/HADOOP/How+To+Contribute for help.
Subsystem Report/Notes
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8076/2/console
versions git=2.34.1
Powered by Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@slfan1989
Copy link
Contributor

@brumi1024 This looks really cool—thanks for your contribution!

@brumi1024
Copy link
Member Author

Thanks @slfan1989 😊

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 0s Docker mode activated.
-1 ❌ patch 0m 40s #8076 does not apply to trunk. Rebase required? Wrong Branch? See https://cwiki.apache.org/confluence/display/HADOOP/How+To+Contribute for help.
Subsystem Report/Notes
GITHUB PR #8076
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8076/9/console
versions git=2.34.1
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

@K0K0V0K
Copy link
Contributor

K0K0V0K commented Dec 1, 2025

Thanks @brumi1024 for this new feature!
LGTM

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 0s Docker mode activated.
-1 ❌ patch 0m 57s #8076 does not apply to trunk. Rebase required? Wrong Branch? See https://cwiki.apache.org/confluence/display/HADOOP/How+To+Contribute for help.
Subsystem Report/Notes
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8076/10/console
versions git=2.34.1
Powered by Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@brumi1024
Copy link
Member Author

The commit is too big for it to be successfully built. Locally the build works even with the updated yarn-ui target (-Pyarn-ui), and since this commit is just about moving the code and ensuring it's built I think we shouldn't block it on yetus. Merging it to trunk, and then I'll be monitoring the builds if any issue occurs.

@brumi1024 brumi1024 merged commit 1995d34 into apache:trunk Dec 1, 2025
1 of 3 checks passed
@brumi1024 brumi1024 deleted the YARN-11886 branch December 1, 2025 15:03
@steveloughran
Copy link
Contributor

ok need some license fixes.

hadoop-yarn-project/hadoop-yarn/hadoop-yarn-capacity-scheduler-ui/src/main/webapp/src/features/queue-management/components/CapacityRowEditor.tsx:1:Missing Apache License
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-capacity-scheduler-ui/src/main/webapp/src/features/queue-management/components/VectorCapacityEditor.tsx:1:Missing Apache License
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-capacity-scheduler-ui/src/main/webapp/src/features/queue-management/components/QueueVectorCapacityDisplay.tsx:1:Missing Apache License
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-capacity-scheduler-ui/src/main/webapp/src/features/queue-management/components/QueueValidationBadges.tsx:1:Missing Apache License
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-capacity-scheduler-ui/src/main/webapp/src/features/queue-management/components/QueueCardContextMenu.tsx:1:Missing Apache License
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-capacity-scheduler-ui/src/main/webapp/src/features/queue-management/components/RemainingHelperDisplay.tsx:1:Missing Apache License
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-capacity-scheduler-ui/src/main/webapp/src/features/property-editor/components/CapacityPropertyField.tsx:1:Missing Apache License
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-capacity-scheduler-ui/src/main/webapp/src/features/property-editor/components/ValidationIssuesPopover.tsx:1:Missing Apache License
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-capacity-scheduler-ui/src/main/webapp/src/features/property-editor/components/EnumPropertyField.tsx:1:Missing Apache License
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-capacity-scheduler-ui/src/main/webapp/src/features/property-editor/components/PropertyFieldHelpers.tsx:1:Missing Apache License
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-capacity-scheduler-ui/src/main/webapp/src/features/queue-comparison/components/__tests__/QueueComparisonDialog.test.tsx:1:Missing Apache License
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-capacity-scheduler-ui/src/main/webapp/src/features/validation/README.md:1:Missing Apache License
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-capacity-scheduler-ui/src/main/webapp/src/app/app.css:1:Missing Apache License
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-capacity-scheduler-ui/src/main/webapp/eslint.config.js:1:Missing Apache License
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-capacity-scheduler-ui/README.md:1:Missing Apache License

I will do the (am doing) the hadoop-assemblies failure #8112 ; will leave this one to you.

@K0K0V0K @brumi1024
would be nice if one of you could approve my PR...

@brumi1024
Copy link
Member Author

@steveloughran yes, sorry I missed those. Approved your PR, and will create a follow-up to fix the licence issues.

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.

5 participants