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
Bump feign timeout #2491
Bump feign timeout #2491
Conversation
// add some padding to the feign timeout, as in many cases lock requests default to a 60 second timeout, | ||
// and we don't want it to exactly align with the feign timeout | ||
private static final Request.Options DEFAULT_FEIGN_OPTIONS = new Request.Options( | ||
10_000, 65_000); |
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.
I guess we don't have a reference to the default lock request timeout (AtlasDbConstants.DEFAULT_TRANSACTION_LOCK_ACQUIRE_TIMEOUT_MS
) so it's inconvenient to try to perform math on that.
Codecov Report
@@ Coverage Diff @@
## develop #2491 +/- ##
==========================================
Coverage ? 60.02%
Complexity ? 4656
==========================================
Files ? 860
Lines ? 40112
Branches ? 4070
==========================================
Hits ? 24077
Misses ? 14567
Partials ? 1468
Continue to review full report at Codecov.
|
Goals (and why):
Fixes #2455
Implementation Description (bullets):
Concerns (what feedback would you like?):
Where should we start reviewing?:
Priority (whenever / two weeks / yesterday):
This change is