phet commented on code in PR #3950:
URL: https://github.com/apache/gobblin/pull/3950#discussion_r1605576277


##########
gobblin-service/src/main/java/org/apache/gobblin/service/modules/orchestration/DagActionStore.java:
##########
@@ -30,12 +30,14 @@
 public interface DagActionStore {
   public static final String NO_JOB_NAME_DEFAULT = "";
   enum DagActionType {
+    CANCEL, // Invoked through DagManager if flow has been stuck in 
Orchestrated state for a while
     KILL, // Kill invoked through API call
-    RESUME, // Resume flow invoked through API call
     LAUNCH, // Launch new flow execution invoked adhoc or through scheduled 
trigger
+    REEVALUATE, // Re-evaluate what needs to be done upon receipt of a final 
job status
+    RESUME, // Resume flow invoked through API call
     RETRY, // Invoked through DagManager for flows configured to allow retries
-    CANCEL, // Invoked through DagManager if flow has been stuck in 
Orchestrated state for a while
-    REEVALUATE // Re-evaluate what needs to be done upon receipt of a final 
job status
+    ENFORCE_START_DEADLINE, // Enforce job start deadline
+    ENFORCE_FINISH_DEADLINE, // Enforce job finish deadline

Review Comment:
   are you sure it's really "job finish"?  somehow I had it in my head that 
it's "flow finish/completion"



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