[
https://issues.apache.org/jira/browse/YARN-10641?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17291708#comment-17291708
]
Andras Gyori commented on YARN-10641:
-------------------------------------
Hi [~zhuqi], thank you for the patch. I always like the idea of making the code
cleaner and fixing bugs in the meanwhile. The patch looks good to me, I have
only minor additions:
* I think testLeafQueueMaxAppUpdateWhenAutoCreation should be named
testAutoQueueCreationMaxAppUpdate (this is the convention other methods are
using)
* The assertEquals method is used the other way around. The first value is the
expected value, the second is the actual.
* LeafQueue has a TODO comment with a typo.
One thing I realised while reading your comment is that we are indeed ignoring
labeling while setting the maximum applications in a queue. As I see it, we
overwrite the maximumApplications value each time while iterating through
labels, which effectively makes the last label to dominate in the calculation.
Perhaps its worth investigating a bit.
> Refactor the max app related update, and fix maxApllications update error
> when add new queues.
> ----------------------------------------------------------------------------------------------
>
> Key: YARN-10641
> URL: https://issues.apache.org/jira/browse/YARN-10641
> Project: Hadoop YARN
> Issue Type: Sub-task
> Reporter: Qi Zhu
> Assignee: Qi Zhu
> Priority: Critical
> Attachments: YARN-10641.001.patch, YARN-10641.002.patch,
> YARN-10641.003.patch, YARN-10641.004.patch,
> image-2021-02-20-15-49-58-677.png, image-2021-02-20-15-53-51-099.png,
> image-2021-02-20-15-55-44-780.png, image-2021-02-20-16-29-18-519.png,
> image-2021-02-20-16-31-13-714.png
>
>
> When refactor the update logic in YARN-10504 .
> The update max applications based abs/cap is wrong, this should be fixed,
> because the max applications is key part to limit applications in CS.
> For example:
> When adding a dynamic queue, the other children's max app of parent queue are
> not updated correctly:
> !image-2021-02-20-15-53-51-099.png|width=639,height=509!
> The new added queue's max app will updated correctly:
> !image-2021-02-20-15-55-44-780.png|width=542,height=426!
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
