[FLINK-33228] Fix the total current resource calculation when fulfilling requirements

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.19.0
• Fix Version/s: 1.19.0
• Component/s: Runtime / Coordination
• Labels: pull-request-available
• Assignee: xiangyu feng
• Reporter: xiangyu feng
• Created: 10/Oct/23 08:02
• Updated: 11/Oct/23 01:59
• Resolved: 11/Oct/23 01:59

Description

Currently, the `totalCurrentResources` calculation in `DefaultResourceAllocationStrategy#tryFulfillRequirements` is wrong.

`ResourceProfile.merge` will not change the original `ResourceProfile`.

Issue Links

• is related to: FLINK-15959 Add min number of slots configuration to limit total number of slots
• links to: GitHub Pull Request #23502