Skip to content

Commit 71c17b9

Browse files
committed
refactor: make DelegatingRetryConfiguration.resolve static
1 parent 98ec565 commit 71c17b9

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

operator-framework-quarkus-extension/deployment/src/main/java/io/javaoperatorsdk/quarkus/extension/deployment/DelegatingRetryConfiguration.java

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,16 +10,20 @@ class DelegatingRetryConfiguration implements RetryConfiguration {
1010

1111
private final RetryConfiguration delegate;
1212

13-
public DelegatingRetryConfiguration(Optional<ExternalRetryConfiguration> retry) {
13+
private DelegatingRetryConfiguration(Optional<ExternalRetryConfiguration> retry) {
1414
delegate =
1515
retry
1616
.<RetryConfiguration>map(ExternalRetryConfigurationAdapter::new)
1717
.orElse(RetryConfiguration.DEFAULT);
1818
}
1919

20-
public RetryConfiguration resolve() {
20+
public static RetryConfiguration resolve(Optional<ExternalRetryConfiguration> retry) {
21+
final var delegate = new DelegatingRetryConfiguration(retry);
2122
return new PlainRetryConfiguration(
22-
getMaxAttempts(), getInitialInterval(), getIntervalMultiplier(), getMaxInterval());
23+
delegate.getMaxAttempts(),
24+
delegate.getInitialInterval(),
25+
delegate.getIntervalMultiplier(),
26+
delegate.getMaxInterval());
2327
}
2428

2529
@Override

operator-framework-quarkus-extension/deployment/src/main/java/io/javaoperatorsdk/quarkus/extension/deployment/QuarkusExtensionProcessor.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -180,7 +180,7 @@ private ControllerConfiguration createControllerConfiguration(
180180
}
181181

182182
private RetryConfiguration retryConfiguration(ExternalControllerConfiguration extConfig) {
183-
return extConfig == null ? null : new DelegatingRetryConfiguration(extConfig.retry).resolve();
183+
return extConfig == null ? null : DelegatingRetryConfiguration.resolve(extConfig.retry);
184184
}
185185

186186
private static class ValueExtractor {

0 commit comments

Comments
 (0)