Github user tillrohrmann commented on a diff in the pull request:
https://github.com/apache/flink/pull/5573#discussion_r175735166
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/job/JobAccumulatorsHandler.java
---
@@ -62,18 +64,33 @@ public JobAccumulatorsHandler(
}
@Override
- protected JobAccumulatorsInfo
handleRequest(HandlerRequest<EmptyRequestBody, JobMessageParameters> request,
AccessExecutionGraph graph) throws RestHandlerException {
- StringifiedAccumulatorResult[] accs =
graph.getAccumulatorResultsStringified();
- List<JobAccumulatorsInfo.UserTaskAccumulator>
userTaskAccumulators = new ArrayList<>(accs.length);
+ protected JobAccumulatorsInfo
handleRequest(HandlerRequest<EmptyRequestBody,
JobAccumulatorsMessageParameters> request, AccessExecutionGraph graph) throws
RestHandlerException {
+ JobAccumulatorsInfo accumulatorsInfo;
+ List<Boolean> queryParams =
request.getQueryParameter(AccumulatorsIncludeSerializedValueQueryParameter.class);
- for (StringifiedAccumulatorResult acc : accs) {
+ boolean includeSerializedValue = false;
--- End diff --
let's make it `final` and assign `false` in the else branch.
---
