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.
---