[
https://issues.apache.org/jira/browse/YARN-9770?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16917058#comment-16917058
]
Haibo Chen commented on YARN-9770:
----------------------------------
Thanks [~jhung] for the patch. The patch looks good to me overall. I have two
minor comments.
1) Can we rename FairQueueOrderingPolicy to RandomQueueOrderingPolicy to reduce
cognitive load as the notion of fairness has been used in FairScheduler for a
different meaning?
2) In the constructor of RandomIterator, given that we kinda assume that the
swap operation is efficient and we are only passing in ArrayList<CSQueue>, how
about we restrict the type to ArrayList<CSQueue>?
The checkstyle issue can also be addressed.
> Create a queue ordering policy which picks child queues with equal probability
> ------------------------------------------------------------------------------
>
> Key: YARN-9770
> URL: https://issues.apache.org/jira/browse/YARN-9770
> Project: Hadoop YARN
> Issue Type: Improvement
> Reporter: Jonathan Hung
> Assignee: Jonathan Hung
> Priority: Major
> Labels: release-blocker
> Attachments: YARN-9770.001.patch, YARN-9770.002.patch
>
>
> Ran some simulations with the default queue_utilization_ordering_policy:
> An underutilized queue which receives an application with many (thousands)
> resource requests will hog scheduler allocations for a long time (on the
> order of a minute). In the meantime apps are getting submitted to all other
> queues, which increases activeUsers in these queues, which drops user limit
> in these queues to small values if minimum-user-limit-percent is configured
> to small values (e.g. 10%).
> To avoid this issue, we assign to queues with equal probability, to avoid
> scenarios where queues don't get allocations for a long time.
--
This message was sent by Atlassian Jira
(v8.3.2#803003)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]