Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/5985#discussion_r187381371
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/rest/messages/job/metrics/MetricsAggregationParameter.java
---
@@ -46,6 +46,13 @@ public String convertValueToString(AggregationMode
value) {
return value.name().toLowerCase();
}
+ @Override
+ public String getDescription() {
+ return "Comma-separated list of aggregates which should be
calculated. Available aggregations are " +
+ " \"sum\", \"max\", \"min\" and \"avg\". Unknown
aggregations are ignored. " +
--- End diff --
`Unknown aggregations are ignored.` seems unnecessary.
---
