Skip to content

v3.1: Streamer: allow 2000 unstaked connection slots (backport of #9289)#9301

Merged
alexpyattaev merged 1 commit intov3.1from
mergify/bp/v3.1/pr-9289
Nov 26, 2025
Merged

v3.1: Streamer: allow 2000 unstaked connection slots (backport of #9289)#9301
alexpyattaev merged 1 commit intov3.1from
mergify/bp/v3.1/pr-9289

Conversation

@mergify
Copy link

@mergify mergify bot commented Nov 26, 2025

Problem

  • Currently we have way more connections incoming over TPU than the size of connection table
  • This results in connection churn and overall loss of efficiency due to evictions
  • allow 4000 unstaked connections in TPU #8144 could not be backported, thus redoing it
  • Not bumping all the way to 4000, safer change to 2000 only due to backport

Currently operators may customize this setting via hidden CLI argument --tpu-max-unstaked-connections, though most choose not to. If this change is not desired, it may be reverted at runtime by supplying desired number via CLI.

Summary of Changes

This is on top of #9288

  • Bump default max unstaked connections from 500 to 2000 to allow for more unstaked senders & reduce connection table churn

Testing on a staked mainnet node, shows that 2000 connection entries in the table are sufficient to accommodate current mainnet demand during leader slots (2 leader slots captured in the plot below):
Screenshot 2025-10-20 at 23 57 36


This is an automatic backport of pull request #9289 done by [Mergify](https://mergify.com).

@mergify mergify bot requested a review from a team as a code owner November 26, 2025 13:18
@t-nelson
Copy link

test breakage looks real?

@alexpyattaev
Copy link

Tests red since this lands after another PR (#9300)

@t-nelson
Copy link

ah that one is clear to merge. get it in and rebase here

@t-nelson
Copy link

@Mergifyio rebase

streamer: bump unstaked connection slots 500 -> 2000
(cherry picked from commit 9169c96)
@mergify
Copy link
Author

mergify bot commented Nov 26, 2025

rebase

✅ Branch has been successfully rebased

@t-nelson t-nelson force-pushed the mergify/bp/v3.1/pr-9289 branch from 6640571 to daecdb0 Compare November 26, 2025 17:16
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.2%. Comparing base (a1ff8b7) to head (daecdb0).

Additional details and impacted files
@@           Coverage Diff           @@
##             v3.1    #9301   +/-   ##
=======================================
  Coverage    83.2%    83.2%           
=======================================
  Files         865      865           
  Lines      375623   375623           
=======================================
+ Hits       312674   312694   +20     
+ Misses      62949    62929   -20     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@alexpyattaev alexpyattaev merged commit c103244 into v3.1 Nov 26, 2025
44 checks passed
@alexpyattaev alexpyattaev deleted the mergify/bp/v3.1/pr-9289 branch November 26, 2025 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants