[
https://issues.apache.org/jira/browse/GOBBLIN-1847?focusedWorklogId=867872&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-867872
]
ASF GitHub Bot logged work on GOBBLIN-1847:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 27/Jun/23 21:20
Start Date: 27/Jun/23 21:20
Worklog Time Spent: 10m
Work Description: homatthew commented on code in PR #3711:
URL: https://github.com/apache/gobblin/pull/3711#discussion_r1244374291
##########
gobblin-cluster/src/main/java/org/apache/gobblin/cluster/GobblinHelixJobLauncher.java:
##########
@@ -462,10 +466,14 @@ public void launchJob(@Nullable JobListener jobListener)
throws JobException {
}
// TODO: Better error handling. The current impl swallows exceptions for
jobs that were started by this method call.
- // One potential way to improve the error handling is to make this error
swallowing conifgurable
+ // One potential way to improve the error handling is to make this error
swallowing configurable
} catch (Throwable t) {
errorInJobLaunching = t;
} finally {
+ if (isCancelWorkflowOnExitEnabled) {
+ cancelJob(jobListener);
Review Comment:
For example, we always clean up the Gobblin state at the end of the job in
the finally block. It's not like there's a point to keeping this helix job
around. In the edge case scenario this Helix flow runs, we will cause issues
with ZK / Helix
Issue Time Tracking
-------------------
Worklog Id: (was: 867872)
Time Spent: 1h 10m (was: 1h)
> HelixException from the Job manager in Fast Ingest should try to delete the
> existing workflow
> ---------------------------------------------------------------------------------------------
>
> Key: GOBBLIN-1847
> URL: https://issues.apache.org/jira/browse/GOBBLIN-1847
> Project: Apache Gobblin
> Issue Type: Improvement
> Reporter: Matthew Ho
> Priority: Major
> Time Spent: 1h 10m
> Remaining Estimate: 0h
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
