[
https://issues.apache.org/jira/browse/GOBBLIN-2110?focusedWorklogId=926301&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-926301
]
ASF GitHub Bot logged work on GOBBLIN-2110:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 17/Jul/24 07:22
Start Date: 17/Jul/24 07:22
Worklog Time Spent: 10m
Work Description: pratapaditya04 commented on code in PR #4001:
URL: https://github.com/apache/gobblin/pull/4001#discussion_r1680552636
##########
gobblin-utility/src/main/java/org/apache/gobblin/util/retry/RetryerFactory.java:
##########
@@ -53,10 +59,14 @@ public class RetryerFactory<T> {
// value large or equal to 1
public static final String RETRY_TIMES = "retry_times";
- private static final Predicate<Throwable> RETRY_EXCEPTION_PREDICATE;
+ // Key to store a comma-separated list of exception class names that should
be retried
+ public static final String EXCEPTION_LIST_FOR_RETRY_CONFIG_KEY =
"EXCEPTION_LIST_FOR_RETRY";
Review Comment:
makes sense, addressed
##########
gobblin-utility/src/main/java/org/apache/gobblin/util/retry/RetryerFactory.java:
##########
@@ -87,15 +97,18 @@ public static <T> Retryer<T> newInstance(Config config,
Optional<RetryListener>
RetryType type =
RetryType.valueOf(config.getString(RETRY_TYPE).toUpperCase());
RetryerBuilder<T> builder;
+
+ Predicate<Throwable> retryPredicate =
getRetryPredicateFromConfigOrDefault(config);
+
switch (type) {
case EXPONENTIAL:
- builder = newExponentialRetryerBuilder(config);
+ builder = newExponentialRetryerBuilder(config, retryPredicate);
break;
case FIXED:
- builder = newFixedRetryerBuilder(config);
+ builder = newFixedRetryerBuilder(config, retryPredicate);
Review Comment:
addressed
Issue Time Tracking
-------------------
Worklog Id: (was: 926301)
Time Spent: 2h (was: 1h 50m)
> Make retry_exception_predicate configurable in RetryerFactory
> -------------------------------------------------------------
>
> Key: GOBBLIN-2110
> URL: https://issues.apache.org/jira/browse/GOBBLIN-2110
> Project: Apache Gobblin
> Issue Type: Bug
> Reporter: Aditya Pratap Singh
> Priority: Minor
> Time Spent: 2h
> Remaining Estimate: 0h
>
> Make retry_exception_predicate configurable in RetryerFactory
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
