[jira] [Work logged] (GOBBLIN-2022) create dag proc for taking actions on job completion

[ASF GitHub Bot (Jira)]

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

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

                Author: ASF GitHub Bot
            Created on: 26/Mar/24 01:25
            Start Date: 26/Mar/24 01:25
    Worklog Time Spent: 10m 
      Work Description: arjun4084346 commented on code in PR #3896:
URL: https://github.com/apache/gobblin/pull/3896#discussion_r1538469764


##########
gobblin-service/src/main/java/org/apache/gobblin/service/modules/orchestration/proc/DagProc.java:
##########
@@ -46,18 +50,21 @@ public final void process(DagManagementStateStore 
dagManagementStateStore) throw
     S state = initialize(dagManagementStateStore);   // todo - retry
     T result = act(dagManagementStateStore, state);   // todo - retry
     commit(dagManagementStateStore, result);   // todo - retry
-    sendNotification(result, eventSubmitter);   // todo - retry
     log.info("{} successfully concluded actions for dagId : {}", 
getClass().getSimpleName(), getDagId());
   }
 
-  protected abstract DagManager.DagId getDagId();
+  protected DagManager.DagId getDagId() {
+    return this.dagTask.getDagId();
+  }
+
+  protected DagNodeId getDagNodeId() {
+    return this.dagTask.getDagNodeId();
+  }
 
   protected abstract S initialize(DagManagementStateStore 
dagManagementStateStore) throws IOException;
 
   protected abstract T act(DagManagementStateStore dagManagementStateStore, S 
state) throws IOException;
 
-  protected abstract void sendNotification(T result, EventSubmitter 
eventSubmitter) throws IOException;

Review Comment:
   events are being emitted as soon as they need to be. in past (pr#3641) it 
was observed that late emission can create problems





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

    Worklog Id:     (was: 911440)
    Time Spent: 2.5h  (was: 2h 20m)

> create dag proc for taking actions on job completion
> ----------------------------------------------------
>
>                 Key: GOBBLIN-2022
>                 URL: https://issues.apache.org/jira/browse/GOBBLIN-2022
>             Project: Apache Gobblin
>          Issue Type: Task
>            Reporter: Arjun Singh Bora
>            Priority: Major
>          Time Spent: 2.5h
>  Remaining Estimate: 0h
>




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