Github user tillrohrmann commented on a diff in the pull request: https://github.com/apache/flink/pull/5309#discussion_r163905384 --- Diff: flink-runtime/src/test/java/org/apache/flink/runtime/rest/handler/legacy/ExecutionGraphCacheTest.java --- @@ -228,10 +246,10 @@ public void testConcurrentAccess() throws Exception { Collection<AccessExecutionGraph> allExecutionGraphs = allExecutionGraphFutures.get(); --- End diff -- True, thanks for the info. Didn't know `ExecutorCompletionService` before.
---