Skip to content

feat(loadtest): add pool_check.py to verify shared sync connection pool #822

Merged
yoon-park-rl merged 5 commits into
mainfrom
yoon/pool-check
Jul 13, 2026
Merged

feat(loadtest): add pool_check.py to verify shared sync connection pool #822
yoon-park-rl merged 5 commits into
mainfrom
yoon/pool-check

Conversation

@yoon-park-rl

@yoon-park-rl yoon-park-rl commented Jul 13, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Adds pool_check.py, a no-API-key sanity check that confirms multiple sync Runloop instances reuse a single underlying httpx transport object, preventing file-descriptor exhaustion when many clients are instantiated.

Usage

uv run python loadtest/pool_check.py

Example output

FD limit: 1048576
Creating 20 Runloop instances...

Distinct transport objects across 20 instances: 1
PASS — all instances share one transport (connection pool).

Done.

yoon-park-rl and others added 2 commits July 12, 2026 15:28
@yoon-park-rl @claude
feat(loadtest): add pool_check.py to verify shared sync connection pool
e50d991 
Creates N Runloop instances and asserts they all reference the same
underlying httpx transport object, confirming no per-instance FD growth.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@yoon-park-rl @claude
docs(loadtest): add pool_check.py to README and remove dead open_conn…
6b22339 
…ections_to helper

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@yoon-park-rl
yoon-park-rl requested a review from jrvb-rl July 13, 2026 15:49
yoon-park-rl and others added 3 commits July 13, 2026 10:18
@yoon-park-rl @claude
fix(loadtest): resolve pyright lint errors in pool_check.py
19a23a8 
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@yoon-park-rl @claude
fix(loadtest): sort imports in pool_check.py
e3ce77a 
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@yoon-park-rl @claude
fix(loadtest): format pool_check.py
c2c1b1a 
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@yoon-park-rl
yoon-park-rl merged commit 0000c52 into main Jul 13, 2026
7 checks passed
@yoon-park-rl
yoon-park-rl deleted the yoon/pool-check branch July 13, 2026 17:40
@stainless-app stainless-app Bot mentioned this pull request Jul 13, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jrvb-rl jrvb-rl jrvb-rl approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@yoon-park-rl @jrvb-rl