[ https://issues.apache.org/jira/browse/FLINK-2388?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14636764#comment-14636764 ]
ASF GitHub Bot commented on FLINK-2388: --------------------------------------- Github user mxm commented on a diff in the pull request: https://github.com/apache/flink/pull/930#discussion_r35204559 --- Diff: flink-runtime/src/main/java/org/apache/flink/runtime/executiongraph/ExecutionGraph.java --- @@ -580,6 +581,34 @@ public ExecutionContext getExecutionContext() { return result; } + /** + * Returns the a stringified version of the user-defined accumulators. + * @return an Array containing the StringifiedAccumulatorResult objects + */ + public StringifiedAccumulatorResult[] getAccumulatorResultsStringified() { + + Map<String, Accumulator<?, ?>> accumulatorMap = aggregateUserAccumulators(); + + int num = accumulatorMap.size(); + StringifiedAccumulatorResult[] resultStrings = new StringifiedAccumulatorResult[num]; + + int i = 0; + for (Map.Entry<String, Accumulator<?, ?>> entry : accumulatorMap.entrySet()) { + + StringifiedAccumulatorResult result; + Accumulator<?, ?> value = entry.getValue(); + if (value != null) { + result = new StringifiedAccumulatorResult(entry.getKey(), value.getClass().getSimpleName(), value.toString()); + } else { + result = new StringifiedAccumulatorResult(entry.getKey(), "null", "null"); --- End diff -- StringifiedAccumulatorResult expects a String. I thought it's better to pass a "null" String instead of null. > JobManager should try retrieving jobs from archive > -------------------------------------------------- > > Key: FLINK-2388 > URL: https://issues.apache.org/jira/browse/FLINK-2388 > Project: Flink > Issue Type: Task > Components: JobManager > Reporter: Enrique Bautista Barahona > > I was following the quickstart guide with the WordCount example and, when I > entered the "analyze" page for the job, nothing came up. Apparently the > JobManagerInfoServlet fails with a NullPointerException. > I've been reading the code and I've seen the problem is in the processing of > the RequestAccumulatorResultsStringified message in JobManager. There's a > TODO where the accumulators should be retrieved from the archive. > As I wanted to know more about Flink internals, I decided to try and fix it. > I've later seen that there's currently ongoing work in that part of the code, > so I guess maybe it's not needed, but if you want I could submit a PR. If you > have already taken it into account and will solve it shortly, please feel > free to close the issue. > If you want to take a look, the commit is here: > https://github.com/ebautistabar/flink/commit/8536352b21fb6c78ad8840b0397509df04358c6b -- This message was sent by Atlassian JIRA (v6.3.4#6332)