[YARN-9431] Fix flaky junit test fair.TestAppRunnability after YARN-8967

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.3.0
• Fix Version/s: 3.3.0
• Component/s: fairscheduler, test
• Assignee: Wilfred Spiegelenburg
• Reporter: Wilfred Spiegelenburg
• Created: 01/Apr/19 02:01
• Updated: 26/Feb/20 05:29
• Resolved: 01/Apr/19 18:23

Description

In YARN-4901 one of the scheduler tests failed. This seems to be linked to the changes around the placement rules introduced in YARN-8967.

Applications submitted in the tests are accepted and rejected at the same time:

2019-04-01 12:00:57,269 INFO  [main] fair.FairScheduler (FairScheduler.java:addApplication(540)) - Accepted application application_0_0001 from user: user1, in queue: root.user1, currently num of applications: 1
2019-04-01 12:00:57,269 INFO  [AsyncDispatcher event handler] fair.FairScheduler (FairScheduler.java:rejectApplicationWithMessage(1344)) - Reject application application_0_0001 submitted by user user1 application rejected by placement rules.

This should never happen and is most likely due to the way the tests generates the application and events.

Issue Links

• is caused by: YARN-8967 Change FairScheduler to use PlacementRule interface
• relates to: YARN-4901 QueueMetrics needs to be cleared before MockRM is initialized