Skip to content

YARN-11981: Utilise leaf-template capacity for the auto created queue - #8674

Open
Hean-Chhinling wants to merge 1 commit into
apache:trunkfrom
Hean-Chhinling:trunk_YARN-11981
Open

YARN-11981: Utilise leaf-template capacity for the auto created queue#8674
Hean-Chhinling wants to merge 1 commit into
apache:trunkfrom
Hean-Chhinling:trunk_YARN-11981

Conversation

@Hean-Chhinling

@Hean-Chhinling Hean-Chhinling commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Contains content generated by Cursor.

Description of PR

Utilise leaf-template capacity for displaying the capacity of auto created queue.

There are two different type of leaf-template capacity configurations:

  1. In Weight mode -> Flexible Mode
<property>
<name>yarn.scheduler.capacity.root.default.auto-queue-creation-v2.leaf-template.capacity</name>
<value>3w</value>
</property>
  1. In relative and absolute mode -> Legacy mode
<property>
<name>yarn.scheduler.capacity.root.default.leaf-queue-template.capacity</name>
<value>100</value>
</property>

Therefore, the PR handle both of this cases and taking the capacity to the auto created child queue.

How was this patch tested?

Tested locally and works as expected. Please refer to the screenshot below:
Screenshot 2026-08-12 at 10 02 26

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?

AI Tooling

If an AI tool was used:

@hadoop-yetus

Copy link
Copy Markdown

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 51s 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.
_ trunk Compile Tests _
+1 💚 mvninstall 48m 21s trunk passed
+1 💚 shadedclient 83m 9s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+1 💚 mvninstall 1m 28s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 shadedclient 33m 6s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 asflicense 0m 48s The patch does not generate ASF License warnings.
121m 27s
Subsystem Report/Notes
Docker ClientAPI=1.55 ServerAPI=1.55 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8674/1/artifact/out/Dockerfile
GITHUB PR #8674
Optional Tests dupname asflicense shadedclient codespell detsecrets
uname Linux 30ed9ccb92c6 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 / 9dc9ba7
Max. process+thread count 589 (vs. ulimit of 10000)
modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-capacity-scheduler-ui U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-capacity-scheduler-ui
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8674/1/console
versions git=2.43.0 maven=3.9.15
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

@K0K0V0K

K0K0V0K commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Thanks @Hean-Chhinling for fixing this bug!

If i see correctly you checked in the

The PR includes the phrase "Contains content generated by "
where is the name of the AI tool used.

field on the PR, but i dont see this in github. Do i miss something?

@K0K0V0K K0K0V0K 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 @Hean-Chhinling !

LGTM!

@Hean-Chhinling

Copy link
Copy Markdown
Contributor Author

Thanks @Hean-Chhinling for fixing this bug!

If i see correctly you checked in the

The PR includes the phrase "Contains content generated by "
where is the name of the AI tool used.

field on the PR, but i dont see this in github. Do i miss something?

Thanks you for the review @K0K0V0K.
I missed that statement now I added it at the top of my PR.

@susheelgupta7 susheelgupta7 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.

LGTM.

@Hean-Chhinling

Copy link
Copy Markdown
Contributor Author

Hello @brumi1024 and @slfan1989,
May I ask for you guys review here?

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.

4 participants