arjun4084346 commented on PR #4029: URL: https://github.com/apache/gobblin/pull/4029#issuecomment-2302992856
> Need to test this thoroughly with unit tests, I believe this can cause the JobStatusMonitor to have some issues since it doesn't have SKIPPED in the JobStatusMonitor ordered execution statuses? This change is not emitting SKIPPED event, so JobStatusMonitor will never get any such event. It is just updating the status in DagNode. But yes, let me update the code to actually emit the event so `get job status` call show the job status as SKIPPED; and incorporate it in ORDERED_EXECUTION_STATUS -- 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]
