[
https://issues.apache.org/jira/browse/GOBBLIN-2132?focusedWorklogId=930322&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-930322
]
ASF GitHub Bot logged work on GOBBLIN-2132:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 15/Aug/24 05:43
Start Date: 15/Aug/24 05:43
Worklog Time Spent: 10m
Work Description: phet commented on code in PR #4026:
URL: https://github.com/apache/gobblin/pull/4026#discussion_r1717910248
##########
gobblin-service/src/main/java/org/apache/gobblin/service/modules/orchestration/proc/LaunchDagProc.java:
##########
@@ -26,18 +26,26 @@
import lombok.extern.slf4j.Slf4j;
import org.apache.gobblin.configuration.ConfigurationKeys;
+import org.apache.gobblin.metrics.event.TimingEvent;
import org.apache.gobblin.runtime.api.FlowSpec;
import org.apache.gobblin.runtime.api.SpecNotFoundException;
import org.apache.gobblin.service.modules.flowgraph.Dag;
import
org.apache.gobblin.service.modules.orchestration.DagManagementStateStore;
+import org.apache.gobblin.service.modules.orchestration.DagManager;
+import org.apache.gobblin.service.modules.orchestration.DagManagerUtils;
import
org.apache.gobblin.service.modules.orchestration.task.DagProcessingEngineMetrics;
import org.apache.gobblin.service.modules.orchestration.task.LaunchDagTask;
import org.apache.gobblin.service.modules.spec.JobExecutionPlan;
import
org.apache.gobblin.service.modules.utils.FlowCompilationValidationHelper;
/**
- * An implementation for {@link DagProc} that launches a new job.
+ * An implementation for {@link DagProc} that launches the start job of a flow.
+ * If there are multiple start jobs for the flow, {@link ReevaluateDagProc} is
created for each of them and that
+ * launches those start jobs.
+ * In a life cycle of a flow, {@link LaunchDagProc} runs only one time, unless
it fails and is
+ * retried by the retry-reminders. {@link ReevaluateDagProc} runs multiple
times depending upon the number of jobs and
+ * number of parallel jobs.
Review Comment:
suggest for final sentence:
> Post-launch, a subsequent {@link ReevaluateDagProc} runs after each job of
the DAG completes. That then may launch further jobs or conclude execution of
the overall DAG (flow).
Issue Time Tracking
-------------------
Worklog Id: (was: 930322)
Time Spent: 1h 20m (was: 1h 10m)
> set and emit flow running status
> --------------------------------
>
> Key: GOBBLIN-2132
> URL: https://issues.apache.org/jira/browse/GOBBLIN-2132
> Project: Apache Gobblin
> Issue Type: Bug
> Reporter: Arjun Singh Bora
> Priority: Major
> Time Spent: 1h 20m
> Remaining Estimate: 0h
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
