flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From aljoscha <...@git.apache.org>
Subject [GitHub] flink issue #5100: [FLINK-4812][metrics] Expose currentLowWatermark for all ...
Date Wed, 29 Nov 2017 17:03:38 GMT
Github user aljoscha commented on the issue:

    https://github.com/apache/flink/pull/5100
  
    I think the fact the the operator needs this in `setup()` is already problematic, the
latency metrics implementation shouldn't have been merged like this. Wouldn't we need the
output gauge only in `OperatorChain` because that creates the `Output`s that operators use
to emit watermarks?


---

Mime
View raw message