Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/4647#discussion_r137093260
--- Diff:
flink-runtime-web/src/main/java/org/apache/flink/runtime/webmonitor/utils/MutableIOMetrics.java
---
@@ -79,6 +85,7 @@ public void addIOMetrics(AccessExecution attempt,
@Nullable MetricFetcher fetche
this.numBytesOut +=
Long.valueOf(metrics.getMetric(MetricNames.IO_NUM_BYTES_OUT, "0"));
this.numRecordsIn +=
Long.valueOf(metrics.getMetric(MetricNames.IO_NUM_RECORDS_IN, "0"));
this.numRecordsOut +=
Long.valueOf(metrics.getMetric(MetricNames.IO_NUM_RECORDS_OUT, "0"));
+ this.metricsFetched = true;
--- End diff --
Unfortunately, if #4472 goes through that neat property of the
synchronization goes away.
---
