Skip to content

Commit

Permalink
[BACKPORT 2.8][#13195] docdb: Increased retryable_request_timeout_sec…
Browse files Browse the repository at this point in the history
…s to 660 seconds

Summary:
`retryable_request_timeout_secs` should be larger than YQL request timeout to avoid consistency issues like #537.
Separate `ysql_client_read_write_timeout_ms` flag was introduced for YSQL and by default it bumps up `YBClient` timeout to 600 seconds: 2294c84.
Hence we need to bump `retryable_request_timeout_secs` to a larger value.

Original commit: D18171 / 413edd0

Test Plan: Jenkins: hot

Reviewers: bogdan, amitanand

Reviewed By: bogdan, amitanand

Subscribers: ybase, amitanand

Differential Revision: https://phabricator.dev.yugabyte.com/D18244
  • Loading branch information
ttyusupov committed Jul 11, 2022
1 parent 89686b0 commit cb0b7ee
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/yb/consensus/retryable_requests.cc
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@

using namespace std::literals;

DEFINE_int32(retryable_request_timeout_secs, 120,
DEFINE_int32(retryable_request_timeout_secs, 660,
"Amount of time to keep write request in index, to prevent duplicate writes.");
TAG_FLAG(retryable_request_timeout_secs, runtime);

Expand Down

0 comments on commit cb0b7ee

Please sign in to comment.