Details
Description
Currently this is how configured queue ordering policy is determined:
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); }
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.