[
https://issues.apache.org/jira/browse/GOBBLIN-1847?focusedWorklogId=867890&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-867890
]
ASF GitHub Bot logged work on GOBBLIN-1847:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 28/Jun/23 01:04
Start Date: 28/Jun/23 01:04
Worklog Time Spent: 10m
Work Description: homatthew commented on code in PR #3711:
URL: https://github.com/apache/gobblin/pull/3711#discussion_r1244537029
##########
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:
Addressed in latest iteration to avoid unnecessary delete / cancel calls to
Helix as we discussed
Issue Time Tracking
-------------------
Worklog Id: (was: 867890)
Time Spent: 1h 50m (was: 1h 40m)
> 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 50m
> Remaining Estimate: 0h
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
