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
---
