XComp commented on code in PR #21019:
URL: https://github.com/apache/flink/pull/21019#discussion_r992679151
##########
flink-streaming-java/src/main/java/org/apache/flink/streaming/api/graph/StreamingJobGraphGenerator.java:
##########
@@ -841,6 +842,12 @@ private StreamConfig createJobVertex(Integer streamNodeId,
OperatorChainInfo cha
},
serializationExecutor));
}
+ coordinatorSerializationFutures.add(
+ FutureUtils.combineAll(serializationFutures)
+ .thenAccept(
+ serializedCoordinators ->
+ serializedCoordinators.forEach(
+
jobVertex::addOperatorCoordinator)));
Review Comment:
Ok, (as always :innocent: ) forget my previous statement. I misunderstood
your concern initially. I pushed another proposal in
383bb9380f82a97cb8ae3d211add64ffbaa5f023
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
