Skip to content

gh-90622: Do not spawn ProcessPool workers on demand via fork method. #91598

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

Merged
gpshead merged 8 commits into from
May 8, 2022
Merged

gh-90622: Do not spawn ProcessPool workers on demand via fork method. #91598

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
e7e895f
gh-90622: Do not spawn ProcessPool workers on demand when they spawn …
gpshead Apr 16, 2022
2ebd47a
Merge branch 'main' into disallow-dynamic-process-spawn-with-fork-bug…
gpshead Apr 17, 2022
1f63d59
Use single leading _s, disable assertion.
gpshead May 6, 2022
b8039a0
Merge branch 'main' into disallow-dynamic-process-spawn-with-fork-bug…
gpshead May 6, 2022
a6bd0c3
Merge branch 'main' into disallow-dynamic-process-spawn-with-fork-bug…
gpshead May 6, 2022
3f1789a
Skip test_idle_process tests on "fork"
gpshead May 6, 2022
5b3d3a8
Merge branch 'main' into disallow-dynamic-process-spawn-with-fork-bug…
gpshead May 8, 2022
8defbef
Merge branch 'main' into disallow-dynamic-process-spawn-with-fork-bug…
gpshead May 8, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'main' into disallow-dynamic-process-spawn-with-fork-bug… 
…fix-only
  • Loading branch information
@gpshead
gpshead authored Apr 17, 2022
commit 2ebd47ab629723821485fbacad522842cba9bdf8
1 change: 1 addition & 0 deletions Lib/test/test_concurrent_futures.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -502,6 +502,7 @@ def acquire_lock(lock):
expected_num_processes = self.worker_count
else:
expected_num_processes = 3

sem = mp_context.Semaphore(0)
for _ in range(3):
self.executor.submit(acquire_lock, sem)
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.