quic: increase timeout and keep alive

anza-xyz · Jan 29, 2025 · fc4952e · fc4952e
1 parent 278b447
commit fc4952e
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git 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
@@ -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,