New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Core: Default to exponential retry strategy in REST client #8366
Conversation
core/src/main/java/org/apache/iceberg/rest/ExponentialHttpRequestRetryStrategy.java
Show resolved
Hide resolved
86fcb68
to
2c0c7a9
Compare
core/src/main/java/org/apache/iceberg/rest/ExponentialHttpRequestRetryStrategy.java
Outdated
Show resolved
Hide resolved
78449e1
to
adb4636
Compare
@@ -96,6 +99,9 @@ private HTTPClient( | |||
clientBuilder.addRequestInterceptorLast(requestInterceptor); | |||
} | |||
|
|||
int maxRetries = PropertyUtil.propertyAsInt(properties, REST_MAX_RETRIES, 5); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a higher retry count than before with the default (which is 1 retry I believe), just wanted to double check that was the intent.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes this was the intent to increase the retry
c0d59bd
to
095acf0
Compare
095acf0
to
45fcd7d
Compare
Thanks, @nastra! |
This adds a new retry strategy, allowing exponential backoff and also allows to adjust the number of retries for the
DefaultHttpRequestRetryStrategy