[
https://issues.apache.org/jira/browse/YARN-10953?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shilun Fan updated YARN-10953:
------------------------------
Component/s: capacity scheduler
> Make CapacityScheduler#getOrCreateQueueFromPlacementContext easier to
> comprehend
> --------------------------------------------------------------------------------
>
> Key: YARN-10953
> URL: https://issues.apache.org/jira/browse/YARN-10953
> Project: Hadoop YARN
> Issue Type: Sub-task
> Components: capacity scheduler
> Affects Versions: 3.4.0
> Reporter: Szilard Nemeth
> Assignee: Andras Gyori
> Priority: Minor
> Labels: pull-request-available
> Fix For: 3.4.0
>
> Time Spent: 1h 50m
> Remaining Estimate: 0h
>
> 1. Most of the method body is wrapped in an if-statement that checks if the
> queue is null. We could negate this and return immediately if the queue !=
> null, so we don't need a large if statement.
> 2. Similarly in that large if body, there's a check for
> fallbackContext.hasParentQueue(). If it's true, we are having yet another
> large if-body. We should also negate this condition and return immediately if
> it's false.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
