[
https://issues.apache.org/jira/browse/YARN-9824?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16926885#comment-16926885
]
Haibo Chen commented on YARN-9824:
----------------------------------
001 Patch looks good to me. +1 from me.
> Fall back to configured queue ordering policy class name
> --------------------------------------------------------
>
> Key: YARN-9824
> URL: https://issues.apache.org/jira/browse/YARN-9824
> Project: Hadoop YARN
> Issue Type: Improvement
> Reporter: Jonathan Hung
> Assignee: Jonathan Hung
> Priority: Major
> Labels: release-blocker
> Attachments: YARN-9824.001.patch
>
>
> Currently this is how configured queue ordering policy is determined:
> {noformat}
> if (policyType.trim().equals(QUEUE_UTILIZATION_ORDERING_POLICY)) {
> // Doesn't respect priority
> qop = new PriorityUtilizationQueueOrderingPolicy(false);
> } else if (policyType.trim().equals(
> QUEUE_PRIORITY_UTILIZATION_ORDERING_POLICY)) {
> qop = new PriorityUtilizationQueueOrderingPolicy(true);
> } else {
> String message =
> "Unable to construct queue ordering policy=" + policyType + " queue="
> + queue;
> throw new YarnRuntimeException(message);
> } {noformat}
> If we want to enable a policy which is not QUEUE_UTILIZATION_ORDERING_POLICY
> or QUEUE_PRIORITY_UTILIZATION_ORDERING_POLICY, it requires code change here
> to add a keyword for this policy.
> It'd be easier if the admin could configure a class name here instead of
> requiring a keyword.
--
This message was sent by Atlassian Jira
(v8.3.2#803003)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
