[YARN-10277] CapacityScheduler test TestUserGroupMappingPlacementRule should build proper hierarchy

• Type: Task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.4.0, 3.3.1
• Fix Version/s: 3.4.0, 3.3.1
• Component/s: capacity scheduler
• Target Version/s: 3.4.0, 3.3.1
• Hadoop Flags: Reviewed
• Assignee: Szilard Nemeth
• Reporter: Gergely Pollák
• Created: 19/May/20 10:24
• Updated: 12/Feb/24 07:28
• Resolved: 30/Jun/20 09:33

Description

Since the CapacityScheduler internal implementation depends more and more on queue being hierarchical, the test gets really hard to maintain. A lot of test cases were failing because they used non existing queues, but the older placement rule solution ignored missing parents, but since the leaf queue change in CS, we must be able to get a full path for any queue, since all queues are referenced by their full path.

This test should reflect this and instead of creating and expecting the existance of fictional queues, it should create a proper queue hierarchy, with a way to describe it better. 

Currently we set up a bunch of mockito "when" statements to simulate the queue behavior, but this is a hassle to maintain, and easy to miss a few method.

Issue Links

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