Skip to content

Run sync event handlers off the asyncio loop via executor #6518

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
masenf wants to merge 1 commit into
base: main
Choose a base branch
from

feat(event): run non-async handlers in a thread pool by default

984b6b0
Select commit
Loading
Failed to load commit list.
Draft

Run sync event handlers off the asyncio loop via executor #6518

feat(event): run non-async handlers in a thread pool by default
984b6b0
Select commit
Loading
Failed to load commit list.
CodSpeed HQ / CodSpeed Performance Analysis failed May 15, 2026 in 0s

Performance Regression: -7.17%

⚠️ Different runtime environments detected

Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.

Open the report in CodSpeed to investigate

❌ 1 regressed benchmark
✅ 23 untouched benchmarks

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Benchmark BASE HEAD Efficiency
❌ test_process_event 7.5 ms 8.1 ms -7.17%

Tip

Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.

Comparing claude/add-event-executor-option-grkJH (984b6b0) with main (9ed3692)

Open in CodSpeed
View more details on CodSpeed HQ