Skip to content

Commit

Permalink
session: increase default tracing fetch attempts
Browse files Browse the repository at this point in the history
Default values are a bit optimistic: fetching will be attempted 5 times,
with 3ms sleeps between attempts. I am reluctant to increase sleep time,
because it will increase latency in optimistic case.

This commit increases default attempts number from 5 to 10, so that
fetching tracing is more reliable, bot not overly lengthy.
  • Loading branch information
Lorak-mmk committed Mar 19, 2024
1 parent 12aa7ea commit d0e8ca2
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion scylla/src/transport/session.rs
Original file line number Diff line number Diff line change
Expand Up @@ -329,7 +329,7 @@ impl SessionConfig {
#[cfg(feature = "cloud")]
cloud_config: None,
enable_write_coalescing: true,
tracing_info_fetch_attempts: NonZeroU32::new(5).unwrap(),
tracing_info_fetch_attempts: NonZeroU32::new(10).unwrap(),
tracing_info_fetch_interval: Duration::from_millis(3),
tracing_info_fetch_consistency: Consistency::One,
cluster_metadata_refresh_interval: Duration::from_secs(60),
Expand Down

0 comments on commit d0e8ca2

Please sign in to comment.