Skip to content

Commit b208317

Browse files
committed
readyset-server: Make table request timeout configurable
Expose domain table request timeout as a CLI/env option, mirroring the existing worker request timeout. Release-Note-Core: Added CLI/env option TABLE_REQUEST_TIMEOUT_SECONDS to configure timeout for long table operations like compaction. AI-Use: level:3 Change-Id: I1d0476b84372a415bb1047b533ac775c6a6a6964 Reviewed-on: https://gerrit.readyset.name/c/readyset/+/11085 Reviewed-by: Gautam Gopinadhan <[email protected]> Tested-by: Buildkite CI
1 parent 141279b commit b208317

File tree

2 files changed

+10
-0
lines changed

2 files changed

+10
-0
lines changed

readyset-server/src/builder.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -101,6 +101,7 @@ impl Builder {
101101
builder.set_post_lookup(opts.feature_post_lookup);
102102
builder.set_parsing_preset(opts.parsing_preset);
103103
builder.set_worker_timeout(Duration::from_secs(opts.worker_request_timeout_seconds));
104+
builder.set_table_request_timeout(Duration::from_secs(opts.table_request_timeout_seconds));
104105
builder.set_background_recovery_interval(Duration::from_secs(
105106
opts.background_recovery_interval_seconds,
106107
));

readyset-server/src/lib.rs

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -623,6 +623,15 @@ pub struct WorkerOptions {
623623
)]
624624
pub worker_request_timeout_seconds: u64,
625625

626+
/// Timeout in seconds for table requests issued to domains
627+
#[arg(
628+
long,
629+
env = "TABLE_REQUEST_TIMEOUT_SECONDS",
630+
default_value = "1800",
631+
hide = true
632+
)]
633+
pub table_request_timeout_seconds: u64,
634+
626635
/// Interval, in seconds, on which to automatically run recovery as long as there are
627636
/// unscheduled domains
628637
#[arg(

0 commit comments

Comments
 (0)