abhishekmjain commented on code in PR #4174:
URL: https://github.com/apache/gobblin/pull/4174#discussion_r2916449389
##########
gobblin-temporal/src/main/java/org/apache/gobblin/temporal/ddm/activity/impl/ProcessWorkUnitImpl.java:
##########
@@ -91,7 +94,16 @@ public int processWorkUnit(WorkUnitClaimCheck wu) {
troubleshooter =
AutomaticTroubleshooterFactory.createForJob(jobState.getProperties());
troubleshooter.start();
return execute(workUnits, wu, jobState, fs,
troubleshooter.getIssueRepository(), jobState.getProperties());
- } catch (IOException | InterruptedException e) {
+ } catch (IOException e) {
+ Throwable cause = e.getCause();
+ if (cause instanceof DSQuotaExceededException || cause instanceof
NSQuotaExceededException) {
Review Comment:
can we create a set of non retryable exceptions and check on that instead of
writing each exception here?
--
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]
