Skip to content

Reject interpreter pool activity executors - #1915

Open
tconley1428 wants to merge 3 commits into
mainfrom
fix/reject-interpreter-pool-activity-executor
Open

tconley1428 wants to merge 3 commits into
mainfrom
fix/reject-interpreter-pool-activity-executor

Conversation

@tconley1428

Copy link
Copy Markdown
Contributor

Passing an InterpreterPoolExecutor as activity_executor currently takes the shared-thread execution path because it subclasses ThreadPoolExecutor, then fails when activity context objects cross interpreter boundaries.

Reject interpreter pools, including subclasses, during worker configuration with a clear ValueError recommending thread or process pools. Guard detection for Python versions before 3.14, document the unsupported executor, and add a changelog entry.

Validation:

  • Both regression cases pass on Python 3.14; the tests skip on older versions.
  • poe lint passes, including all three type checkers.
  • git diff --check passes.

Refs #1154. This provides explicit rejection; it does not implement subinterpreter support.

@tconley1428
tconley1428 requested a review from a team as a code owner October 1, 2026 23:28

This branch has not been deployed

No deployments
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.

2 participants