Github user yew1eb commented on a diff in the pull request:
https://github.com/apache/flink/pull/4869#discussion_r147092709
--- Diff: docs/monitoring/metrics.md ---
@@ -856,66 +918,80 @@ Thus, in order to infer the metric identifier:
<table class="table table-bordered">
<thead>
<tr>
- <th class="text-left" style="width: 20%">Scope</th>
- <th class="text-left" style="width: 30%">Metrics</th>
- <th class="text-left" style="width: 50%">Description</th>
+ <th class="text-left" style="width: 18%">Scope</th>
+ <th class="text-left" style="width: 26%">Metrics</th>
+ <th class="text-left" style="width: 48%">Description</th>
+ <th class="text-left" style="width: 8%">Type</th>
</tr>
</thead>
<tbody>
<tr>
<th rowspan="7"><strong>Task</strong></th>
<td>currentLowWatermark</td>
- <td>The lowest watermark this task has received.</td>
+ <td>The lowest watermark this task has received (in
milliseconds).</td>
--- End diff --
I see that `currentLowWatermark` is derived from the
`Watermark.getTimestamp ()` method,
```
/**
* Returns the timestamp associated with this {@link Watermark} in
milliseconds.
*/
public long getTimestamp() {
return timestamp;
}
```
so `currentLowWatermark` must be milliseconds ?
---