Github user zentol commented on the issue:
https://github.com/apache/flink/pull/5099
@greghogan Thank your for reviewing this ð, will fix the typo while
merging.
---
Github user zentol commented on the issue:
https://github.com/apache/flink/pull/5099
@greghogan I've addressed your comments.
---
Github user zentol commented on the issue:
https://github.com/apache/flink/pull/5099
We could also make `metrics.reporters` an optional list of reporters to
enable; by default we include all reporters we can find. This would be more
user-friendly since you wouldn't have to modify the
Github user zentol commented on the issue:
https://github.com/apache/flink/pull/5099
only checking for the class makes sense.
---
