Github user walterddr commented on a diff in the pull request:
https://github.com/apache/flink/pull/5985#discussion_r187459062
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/rest/messages/AccumulatorsIncludeSerializedValueQueryParameter.java
---
@@ -38,4 +38,10 @@ public String convertValueToString(Boolean value) {
public Boolean convertStringToValue(String value) {
return Boolean.valueOf(value);
}
+
+ @Override
+ public String getDescription() {
+ return "Boolean value. If true then serialized user task
accumulators will be included into a response. " +
+ "False by default.";
--- End diff --
"False by default." is confusing to me, is "false": the default value of
the parameter is `false`, or "false": whether the task accumulator will be
included or not depends on a "default" configuration from the task accumulator?
---
