Github user tillrohrmann commented on a diff in the pull request:
https://github.com/apache/flink/pull/5308#discussion_r163877071
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/executiongraph/ExecutionGraph.java
---
@@ -1701,40 +1705,4 @@ void notifyExecutionChange(
}
}
}
-
- @Override
- public ArchivedExecutionGraph archive() {
--- End diff --
I think it's better that the `ExecutionGraph` does not know about the
`ArchivedExecutionGraph` because the coupling should be vice versa.
---
