Github user GJL commented on the issue: https://github.com/apache/flink/pull/5573 I agree with @aljoscha. We should not add more `public` methods to `ClusterClient`. Implementing `getAccumulators()` in `RestClusterClient` is enough. Regarding the `JobAccumulatorsHandler`, what would speak against adding a new field to `UserTaskAccumulator` which stores the `SerializedValue`? The JSON representation would always carry an additional base64 encoded `byte` array but I think performance isn't important at this point. cc: @tillrohrmann @aljoscha
---