Github user tillrohrmann commented on a diff in the pull request:
https://github.com/apache/flink/pull/4737#discussion_r141613086
--- Diff:
flink-runtime/src/test/java/org/apache/flink/runtime/executiongraph/ArchivedExecutionGraphTest.java
---
@@ -142,6 +143,8 @@ public static void setupExecutionGraph() throws
Exception {
Execution executionWithAccumulators =
runtimeGraph.getJobVertex(v1ID).getTaskVertices()[0].getCurrentExecutionAttempt();
+ runtimeGraph.setJsonPlan("{}");
--- End diff --
This PR introduces null checks in the `ArchivedExecutionGraph` and that's
why this change is included. I could also move these changes to the #4727 PR.
---
