Skip to content

On size10 clusters, the bundled Postgres accepts 500 connections instead of 300 - #3141

Merged
Benjamin Elder (BenTheElder) merged 1 commit into
agent-substrate:mainfrom
haiyanmeng:size10-max-connections-500
Oct 23, 2026
Merged

Benjamin Elder (BenTheElder) merged 1 commit into
agent-substrate:mainfrom
haiyanmeng:size10-max-connections-500

Conversation

@haiyanmeng

@haiyanmeng haiyanmeng commented Oct 22, 2026 •

Copy link
Copy Markdown
Collaborator

size10's max_connections = 300 limits how many ate-api-server replicas can run with a useful pool. Each replica holds its read/write pool plus 6 owner, watch and readiness connections, so eight replicas fit only about 30 pool connections each.

In the cold resume benchmark on the fake data plane (8 replicas at 8 CPUs, size10 Postgres), pool size set the ceiling:

Pool per replica Connections for 8 replicas Ceiling (cycles/s)
30 288 about 6500
55 488 about 7350
80 688 about 7420

AI use: AI-assisted (Claude Code) for the benchmark runs, the analysis and this change.

  • Tests pass
  • Docs updated

At 300, eight ate-api-server replicas fit only about 30 pool connections
each. In the 8 x 8 CPU cold resume benchmark, pools of 30 capped the
ceiling at about 6500 cycles/s. Pools of 55, which need 488 connections,
reached about 7350, and larger pools added nothing.

The comment now counts the owner pool in each replica's connections.

@maxsmythe Max Smythe (maxsmythe) left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

spoke to Eitan Yarmush (@EItanya), we're both OK with this.

@BenTheElder
Benjamin Elder (BenTheElder) added this pull request to the merge queue Oct 23, 2026
Merged via the queue into agent-substrate:main with commit 2b3c60f Oct 23, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants