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.


---

Reply via email to