Skip to content

GH-84559: Deprecate fork being the multiprocessing default. #100618

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 23 commits into from
Feb 2, 2023
Merged

GH-84559: Deprecate fork being the multiprocessing default. #100618

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
1cecbdb
GH-84559: Deprecate fork being the multiprocessing default.
gpshead Dec 30, 2022
df83fba
address CI test failures?
gpshead Dec 31, 2022
ec1e868
the new warning needs no pickle name mapping
gpshead Dec 31, 2022
a1064de
avoid DeprecationWarnings vs -Werror
gpshead Dec 31, 2022
223b748
prefer forkserver over fork for concurrent.futures
gpshead Dec 31, 2022
fc64626
explicitly use spawn for simple process launchers.
gpshead Dec 31, 2022
e45fab6
warn from concurrent.futures, reword things.
gpshead Dec 31, 2022
29764a2
avoid double warning in futures
gpshead Dec 31, 2022
c52b9db
Cleanup messages, test futures warning.
gpshead Dec 31, 2022
4c8cc34
cleanup warn stacklevel logic
gpshead Dec 31, 2022
7179684
Document all the things!
gpshead Dec 31, 2022
f7e3161
Have all doctests use 'spawn'.
gpshead Jan 1, 2023
9947860
Correct warning stacklevel for Manager.
gpshead Jan 1, 2023
5834f2d
more doctest cleanup.
gpshead Jan 1, 2023
7d48e86
NEWS entry.
gpshead Jan 1, 2023
28c0cfc
docs: add the implicit i
gpshead Jan 10, 2023
2820bb8
Merge branch 'main' into multiprocessing/deprecate-fork-default
gpshead Jan 28, 2023
4c16205
Use skip_file_prefixes.
gpshead Jan 28, 2023
7d84a72
use the modern warnings API, doc updates.
gpshead Jan 28, 2023
8b2029e
documentation cleanup
gpshead Jan 28, 2023
846e9a0
Remove an unused variable.
gpshead Feb 1, 2023
7078986
doc typo fix, unnecessary method change revert.
gpshead Feb 1, 2023
e3e3b3c
Add What's New entries.
gpshead Feb 2, 2023
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
documentation cleanup
  • Loading branch information
@gpshead
gpshead committed Jan 28, 2023
commit 8b2029e856c3d8ea19c56d2d10147fa99d18d3b5
6 changes: 3 additions & 3 deletions Doc/library/concurrent.futures.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -288,9 +288,9 @@ to a :class:`ProcessPoolExecutor` will result in deadlock.
.. versionchanged:: 3.12
The implcit use of the :mod:`multiprocessing` *fork* start method as a
Comment thread
gpshead marked this conversation as resolved.
Outdated
platform default (see :ref:`multiprocessing-start-methods`) now raises a
:exc:`DeprecationWarning` as the default will be changing in Python >=
3.14. Code that requires *fork* should explicitly specify that when
creating their ProcessPoolExecutor by passing a
:exc:`DeprecationWarning`. The default will change in Python 3.14.
Code that requires *fork* should explicitly specify that when creating
their :class:`ProcessPoolExecutor` by passing a
``mp_context=multiprocessing.get_context('fork')`` parameter.

.. _processpoolexecutor-example:
Expand Down
14 changes: 7 additions & 7 deletions Doc/library/multiprocessing.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -124,13 +124,7 @@ to start a process. These *start methods* are
inherited by the child process. Note that safely forking a
multithreaded process is problematic.

Available on POSIX systems. Currently the default on POSIX (not macOS).

.. versionchanged:: 3.12
The implicit use of the *fork* start method as the default now raises a
:exc:`DeprecationWarning`. Code that requires it should explicitly
specify *fork* via :func:`get_context` or :func:`set_start_method`.
The default will change in 3.14.
Available on POSIX systems. Currently the default on POSIX except macOS.

*forkserver*
When the program starts and selects the *forkserver* start method,
Expand All @@ -144,6 +138,12 @@ to start a process. These *start methods* are
Available on POSIX platforms which support passing file descriptors
over Unix pipes such as Linux.

.. versionchanged:: 3.12
Implicit use of the *fork* start method as the default now raises a
:exc:`DeprecationWarning`. Code that requires it should explicitly
specify *fork* via :func:`get_context` or :func:`set_start_method`.
The default will change away from *fork* in 3.14.

.. versionchanged:: 3.8

On macOS, the *spawn* start method is now the default. The *fork* start
Expand Down