[FLINK-26117] JobManagerMetricGroup needs to implement GloballyCleanableResource as well

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.15.0
• Fix Version/s: 1.15.0
• Component/s: Runtime / Coordination
• Labels: pull-request-available
• Assignee: Matthias Pohl
• Reporter: Matthias Pohl
• Created: 14/Feb/22 08:58
• Updated: 16/Feb/22 08:52
• Resolved: 16/Feb/22 08:52

Description

There's a mistake being done during the introduction of the *CleanableResources interfaces. The JobManagerMetricGroup should also implement the global cleanup since it's called in both cleanup types not only the local one.

Issue Links

• is caused by: FLINK-25432 Introduce common interfaces for cleaning up local and global job data
• relates to: FLINK-26175 Make local cleanup being called in global cleanup situations
• Testing discovered: FLINK-26122 JobManagerMetricGroup.addJob returns null when already being closed
• Testing discovered: FLINK-26124 JobManagerMetricGroup is never actually used
• links to: GitHub Pull Request #18749