Skip to content

fix(bindings/python): remove greenlet benchmark dependencies - #8189

Merged
Xuanwo merged 1 commit into
mainfrom
xuanwo/python-remove-gevent
Aug 28, 2026
Merged

fix(bindings/python): remove greenlet benchmark dependencies#8189
Xuanwo merged 1 commit into
mainfrom
xuanwo/python-remove-gevent

Conversation

@Xuanwo

@Xuanwo Xuanwo commented Aug 28, 2026

Copy link
Copy Markdown
Member

Which issue does this PR close?

N/A.

Rationale for this change

The Python binding setup installs every dependency group. Its benchmark-only greenify dependency uses CPython's Limited API, which is unsupported by free-threaded Python builds and prevents just setup from completing when uv selects Python 3.14t.

The Boto3 benchmark does not need an alternative I/O runtime. A bounded thread pool preserves its concurrent synchronous workload without coupling the binding development environment to gevent, greenlet, or greenify.

What changes are included in this PR?

Remove the greenlet-based benchmark dependencies and regenerate the uv lockfile. Replace the gevent benchmark with an asyncio-driven bounded thread pool, and update the benchmark commands and documentation.

Are there any user-facing changes?

There are no public API changes. Python binding development setup now works with free-threaded Python, and the Boto3 comparison benchmark uses threads instead of gevent greenlets.

AI Usage Statement

AI materially assisted with dependency analysis, implementation, and validation. The replacement preserves the concurrent workload shape, but no performance-equivalence claim is made because the S3 benchmark was not run against an actual service.

@Xuanwo
Xuanwo marked this pull request as ready for review August 28, 2026 09:08
@dosubot dosubot Bot added size:M This PR changes 30-99 lines, ignoring generated files. releases-note/fix The PR fixes a bug or has a title that begins with "fix" labels Aug 28, 2026
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Aug 28, 2026
@Xuanwo
Xuanwo merged commit d6a4fa3 into main Aug 28, 2026
27 checks passed
@Xuanwo
Xuanwo deleted the xuanwo/python-remove-gevent branch August 28, 2026 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/fix The PR fixes a bug or has a title that begins with "fix" size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants