Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/4647#discussion_r137068223
--- 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 --
Note that this does not necessarily mean that the num/bytes IO metrics have been added
yet; it only means that _some_ metric for the given subtask has already arrived and was inserted
into the store.
---
|