[YUNIKORN-1772] Simplify the cache Context.SelectApplications function

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.3.0
• Component/s: shim - kubernetes
• Labels: pull-request-available
• Target Version: 1.3.0
• Assignee: PoAn Yang
• Reporter: PoAn Yang
• Created: 30/May/23 11:47
• Updated: 20/Jun/23 14:41
• Resolved: 30/May/23 20:27

Description

In production code, we only use cache Context.SelectApplications in KubernetesShim.schedule with a nil filter.

In testing code, MockScheduler.waitAndAssertApplicationState and MockScheduler.waitAndAssertTaskState use it to filter apps by application ID, but this can be replaced by cache Context.GetApplication.

So we can use a new function GetAllApplications, and remove SelectApplications in the code base.

Issue Links

• links to: GitHub Pull Request #602