[FLINK-7876] Register TaskManagerMetricGroup under ResourceID instead of InstanceID

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.4.0
• Fix Version/s: 1.4.0
• Component/s: Runtime / Metrics
• Labels: flip-6
• Assignee: Till Rohrmann
• Reporter: Till Rohrmann
• Created: 19/Oct/17 13:08
• Updated: 01/Nov/17 22:29
• Resolved: 01/Nov/17 22:28

Description

Currently, the TaskManager registers the TaskManagerMetricGroup under its InstanceID and thereby binding its metrics effectively to the lifetime of its registration with the JobManager. This has also implications how the REST handler retrieve the TaskManager metrics, namely by its InstanceID.

I would actually propose to register the TaskManagerMetricGroup under the TaskManager/TaskExecutor ResourceID which is valid over the whole lifetime of the TaskManager/TaskExecutor. That way we would also be able to query metrics independent of the connection status to the JobManager.

Issue Links

• supercedes: FLINK-7100 TaskManager metrics are registered twice
• links to: GitHub Pull Request #4872