[YARN-10280] Find a better way to pass queue manager for UserGroupMappingPlacementRule and AppNameMappingPlacementRule

• Type: Task
• Status: In Progress
• Priority: Major
• Resolution: Unresolved
• Assignee: Gergely Pollák
• Reporter: Gergely Pollák
• Created: 19/May/20 11:53
• Updated: 04/Jun/20 11:11

Description

Since YARN-9789 (Allow multiple leaf queues with the same name in CapacityScheduler) we need to use full queue paths internally, however for backwards compatibility reasons we still allow users to reference queues by their leaf queue name. This  means we need to look up queues in queue manager by their short name to get the full path of the queue. This is why we need the queue manager in certain methods in these classes.

Currently the qm instance is passed via method arguments wherever necessary, but since the Placement rule classes depend on queue manager, it should be passed and stored as a field.

Issue Links

• is depended upon by: YARN-10273 [Umbrella] Followup tasks for CapacityScheduler leaf queue changes (YARN-9879)