GH-84559: Deprecate fork being the multiprocessing default. by gpshead · Pull Request #100618 · python/cpython (original) (raw)
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
[ Show hidden characters]({{ revealButtonHref }})
This starts the process. Users who don't specify their own start method and use the default on platforms where it is 'fork' will see a DeprecationWarning upon multiprocessing.Pool() construction or upon multiprocessing.Process.start().
gpshead marked this pull request as ready for review
gpshead deleted the multiprocessing/deprecate-fork-default branch
gpshead added a commit to gpshead/cpython that referenced this pull request
This reverts the core of python#100618 while leaving relevant documentation improvements and minor refactorings in place.
gpshead added a commit that referenced this pull request
This reverts the core of #100618 while leaving relevant documentation improvements and minor refactorings in place.
russellb added a commit to russellb/vllm that referenced this pull request
The default of fork
is known to be problematic. Python itself is
changing the default to spawn
. The new default is expected to be in
place for Python 3.14.
Python references for the change to the default:
We also have several places where this option had to be set to spawn
to make tests work. The AMD code even checks and overrides the value if
it's not set to spawn
.
Simplify things for everyone and just default to spawn
, but leave the
option in place just in case, at least for now.
Signed-off-by: Russell Bryant rbryant@redhat.com