XComp commented on code in PR #23880:
URL: https://github.com/apache/flink/pull/23880#discussion_r1426546494
##########
flink-runtime/src/main/java/org/apache/flink/runtime/jobmanager/JobGraphWriter.java:
##########
@@ -37,6 +38,18 @@ public interface JobGraphWriter extends
LocallyCleanableResource, GloballyCleana
*/
void putJobGraph(JobGraph jobGraph) throws Exception;
+ /**
+ * Adds the {@link JobGraph} instance and have write operations performed
asynchronously in
+ * ioExecutor of Dispatcher
+ *
+ * @param jobGraph
+ * @param ioExecutor
+ * @return
+ * @throws Exception
+ */
+ CompletableFuture<Void> putJobGraphAsync(JobGraph jobGraph,
Optional<Executor> ioExecutor)
Review Comment:
Can you elaborate a bit more on your claim? Is it because you have to modify
the interface for that change? If you're concerned about chaning the interface:
`JobGraphWriter` is an internally used interface (i.e. not marked in any way as
`@Public` or `@PublicEvolving`). Additionally, it's not really exposed to users
in any way I could think. In this sense, modifying the interface in order to
improve the overall code base (rather than introducing some workarounds) seems
like a reasonable approach. WDYT?
--
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: issues-unsubscr...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
