pytest-dev / pytest-dev/execnet

dropping the execmodel concept again

Open
#150 10 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
102
Forks
47
PR merge metrics
No merged PRs in 30d

Description

CC @ctheune @nicoddemus

the execmodel concept that @hpk42 introduced back in 2013 has multiple drawbacks

  1. it breaks execnet for testing qt, as the main thread is no longer under safe control
  2. it is completely incompatible with the async primitives that are starting to win (asyncio/trio/curio, async/await)

its not quite clear to me for what detailed use-case it was introduced, but as it is right now its a major show-stopper for me to introduce any support for modern async primitives

its the biggest reason why i dropped execnet maintenances, the second biggest reason being that there seemingly where possible placements (unfortunately the replacements turned out not to work out)

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

The issue does not name files, tests, or an entry point for the execmodel concept. Start by locating the execmodel implementation and reviewing the unresolved discussion, then define the affected execnet behavior and compatibility requirements before attempting a change.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
distributed-systems
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.