[ 
https://issues.apache.org/jira/browse/GOBBLIN-1910?focusedWorklogId=898846&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-898846
 ]

ASF GitHub Bot logged work on GOBBLIN-1910:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 10/Jan/24 03:43
            Start Date: 10/Jan/24 03:43
    Worklog Time Spent: 10m 
      Work Description: arjun4084346 commented on code in PR #3853:
URL: https://github.com/apache/gobblin/pull/3853#discussion_r1446848437


##########
gobblin-service/src/main/java/org/apache/gobblin/service/modules/orchestration/DagManagerUtils.java:
##########
@@ -48,25 +54,42 @@
 import org.apache.gobblin.service.modules.flowgraph.Dag.DagNode;
 import 
org.apache.gobblin.service.modules.orchestration.DagManager.FailureOption;
 import org.apache.gobblin.service.modules.spec.JobExecutionPlan;
+import org.apache.gobblin.service.monitoring.JobStatus;
+import org.apache.gobblin.service.monitoring.JobStatusRetriever;
 import org.apache.gobblin.util.ConfigUtils;
 
+import static org.apache.gobblin.service.ExecutionStatus.PENDING;
+
 
 public class DagManagerUtils {
   static long DEFAULT_FLOW_SLA_MILLIS = TimeUnit.HOURS.toMillis(24);
   static String QUOTA_KEY_SEPERATOR = ",";
 
-  static FlowId getFlowId(Dag<JobExecutionPlan> dag) {
+  public static FlowId getFlowId(Dag<JobExecutionPlan> dag) {
     return getFlowId(dag.getStartNodes().get(0));
   }
 
+  public static DagActionStore.DagAction 
createDagActionFromDag(Dag<JobExecutionPlan> dag, DagActionStore.FlowActionType 
flowActionType) {
+    return createDagActionFromDagNode(dag.getStartNodes().get(0), 
flowActionType);
+  }
+
+  // todo - dag action object does not have any identifier to tell if it is 
for a complete dag or just for one dag node

Review Comment:
   looks like `todo` is enough; at least on my intellij





Issue Time Tracking
-------------------

    Worklog Id:     (was: 898846)
    Time Spent: 16h 40m  (was: 16.5h)

> Refactor code to move current in-memory references to new design for REST 
> calls: Launch, Resume and Kill
> --------------------------------------------------------------------------------------------------------
>
>                 Key: GOBBLIN-1910
>                 URL: https://issues.apache.org/jira/browse/GOBBLIN-1910
>             Project: Apache Gobblin
>          Issue Type: New Feature
>            Reporter: Meeth Gala
>            Priority: Major
>          Time Spent: 16h 40m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to