From fc4952e70c86732824dd2a6613aa58fd45d474f9 Mon Sep 17 00:00:00 2001 From: Emanuele Cesena Date: Wed, 29 Jan 2025 16:27:29 -0600 Subject: [PATCH] quic: increase timeout and keep alive --- local-cluster/src/local_cluster_snapshot_utils.rs | 1 + sdk/quic-definitions/src/lib.rs | 8 ++++++-- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/local-cluster/src/local_cluster_snapshot_utils.rs b/local-cluster/src/local_cluster_snapshot_utils.rs index 6240ade176c086..d008781a742f5d 100644 --- a/local-cluster/src/local_cluster_snapshot_utils.rs +++ b/local-cluster/src/local_cluster_snapshot_utils.rs @@ -79,6 +79,7 @@ impl LocalCluster { .rpc_client() .get_slot_with_commitment(CommitmentConfig::processed()) .expect("Couldn't get slot"); + drop(client); // Wait for a snapshot for a bank >= last_slot to be made so we know that the snapshot // must include the transactions just pushed diff --git a/sdk/quic-definitions/src/lib.rs b/sdk/quic-definitions/src/lib.rs index 244073216b49ec..c60373f24e35e1 100644 --- a/sdk/quic-definitions/src/lib.rs +++ b/sdk/quic-definitions/src/lib.rs @@ -16,8 +16,12 @@ pub const QUIC_TOTAL_STAKED_CONCURRENT_STREAMS: usize = 100_000; // forwarded packets from staked nodes. pub const QUIC_MAX_STAKED_CONCURRENT_STREAMS: usize = 512; -pub const QUIC_MAX_TIMEOUT: Duration = Duration::from_secs(2); -pub const QUIC_KEEP_ALIVE: Duration = Duration::from_secs(1); +// Connection idle timeout, and keep alive. +// Quic will close the connection after QUIC_MAX_TIMEOUT, +// and send a ping every QUIC_KEEP_ALIVE. +// These shouldn't be too low to avoid unnecessary ping traffic. +pub const QUIC_MAX_TIMEOUT: Duration = Duration::from_secs(60); +pub const QUIC_KEEP_ALIVE: Duration = Duration::from_secs(45); // Disable Quic send fairness. // When set to false, streams are still scheduled based on priority,