[YUNIKORN-1770] Avoid resource clone in Application.tryNodes()

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.3.0
• Component/s: core - scheduler
• Labels: pull-request-available
• Assignee: Peter Bacsko
• Reporter: Peter Bacsko
• Created: 25/May/23 23:37
• Updated: 20/Jun/23 14:41
• Resolved: 26/May/23 04:43

Description

Similarly to YUNIKORN-1764, we have an unnecessary clone in Application.tryNodes():

score := ask.GetAllocatedResource().FitInScore(node.GetAvailableResource())

This call site was visible in an alloc_space profile that we recently got from an upstream Yunikorn user. We can do this calculation without calling node.GetAvailableResource(), so cloning can be avoided.

Issue Links

• links to: GitHub Pull Request #553