[GitHub] flink pull request #4872: [FLINK-7876] Register TaskManagerMetricGroup under...

[zentol]

Github user zentol commented on a diff in the pull request:

    https://github.com/apache/flink/pull/4872#discussion_r147655034
  
    --- Diff: 
flink-runtime/src/test/java/org/apache/flink/runtime/rest/handler/legacy/metrics/MetricFetcherTest.java
 ---
    @@ -76,25 +74,22 @@ public void testUpdate() throws Exception {
                JobID jobID = new JobID();
                InstanceID tmID = new InstanceID();
    --- End diff --
    
    we no longer need this field; generated the resource ID with 
ResourceID#generate instead.


---