umustafi commented on code in PR #3700:
URL: https://github.com/apache/gobblin/pull/3700#discussion_r1212444910


##########
gobblin-runtime/src/main/java/org/apache/gobblin/runtime/api/DagActionStore.java:
##########
@@ -27,7 +27,9 @@
 public interface DagActionStore {
   enum DagActionValue {
     KILL,
-    RESUME
+    RESUME,
+    // TODO: potentially combine this enum with {@link 
SchedulerLeaseDeterminationStore.FlowActionType}
+    LAUNCH

Review Comment:
   Yes, that is correct. The host that wins the lease is only the one who 
passes that job along to the DagManager to launch (they do this by persisting 
in mysql). In the near future, only the leader host will actually run the job 
but eventually there should be a non-blocking, multi-active DagManager as well 
that will work similarly to win the lease of running the job. 



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

Reply via email to