[issue22087] asyncio: support multiprocessing (support fork)

Yury Selivanov added the comment: New changeset af9cda9845666e2f704177a431d29f91efbf828a by Yury Selivanov in branch '3.7': Revert "bpo-22087: Fix Policy.get_event_loop() to detect fork (GH-7208)" (GH-7233) https://github.com/python/cpython/commit/af9cda9845666e2f704177a431d29f91efbf828a

[issue22087] asyncio: support multiprocessing (support fork)

Yury Selivanov added the comment: New changeset 99279ad823a758288e4e41962abfc4dad8943ce8 by Yury Selivanov in branch 'master': Revert "bpo-22087: Fix Policy.get_event_loop() to detect fork (GH-7208)" (#7232) https://github.com/python/cpython/commit/99279ad823a758288e4e41962abfc4dad8943ce8

[issue22087] asyncio: support multiprocessing (support fork)

Yury Selivanov added the comment: Even though I committed a version of Dan's patch to 3.7 and 3.8, I've finally decided to revert it and do it properly instead. We should strive to implement a proper solution, not commit some half-working code. A concrete plan (for Python 3.8 probably):

[issue22087] asyncio: support multiprocessing (support fork)

Change by Yury Selivanov : -- pull_requests: +6865 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue22087] asyncio: support multiprocessing (support fork)

Change by Yury Selivanov : -- pull_requests: +6864 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue22087] asyncio: support multiprocessing (support fork)

Yury Selivanov added the comment: another related issue: https://bugs.python.org/issue33688 -- ___ Python tracker ___ ___

[issue22087] asyncio: support multiprocessing (support fork)

Change by STINNER Victor : -- nosy: -vstinner ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue22087] asyncio: support multiprocessing (support fork)

Change by Yury Selivanov : -- pull_requests: +6858 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue22087] asyncio: support multiprocessing (support fork)

miss-islington added the comment: New changeset 2a7eb0b531656f4a77d85078e6e009e4b3639ef9 by Miss Islington (bot) in branch '3.7': bpo-22087: Fix Policy.get_event_loop() to detect fork (GH-7208) https://github.com/python/cpython/commit/2a7eb0b531656f4a77d85078e6e009e4b3639ef9 --

[issue22087] asyncio: support multiprocessing (support fork)

Change by Yury Selivanov : -- pull_requests: +6851 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue22087] asyncio: support multiprocessing (support fork)

Change by miss-islington : -- pull_requests: +6847 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue22087] asyncio: support multiprocessing (support fork)

Yury Selivanov added the comment: New changeset 5d97b7bcc19496617bf8c448d2f149cc28c73bc7 by Yury Selivanov in branch 'master': bpo-22087: Fix Policy.get_event_loop() to detect fork (GH-7208) https://github.com/python/cpython/commit/5d97b7bcc19496617bf8c448d2f149cc28c73bc7 --

[issue22087] asyncio: support multiprocessing (support fork)

Change by Yury Selivanov : -- pull_requests: +6843 stage: needs patch -> patch review ___ Python tracker ___ ___ Python-bugs-list

[issue22087] asyncio: support multiprocessing (support fork)

Change by Zac Medico : -- nosy: +zmedico ___ Python tracker ___ ___ Python-bugs-list

[issue22087] asyncio: support multiprocessing (support fork)

Yury Selivanov added the comment: > It seems there's a check in the top-level get_event_loop() function but not > in the DefaultEventLoopPolicy.get_event_loop() method. Yes, as users aren't supposed to work with policies directly. > (also: wow, that stuff is complicated) Yep. Please assign

[issue22087] asyncio: support multiprocessing (support fork)

Antoine Pitrou added the comment: It seems there's a check in the top-level get_event_loop() function but not in the DefaultEventLoopPolicy.get_event_loop() method. (also: wow, that stuff is complicated) -- ___ Python tracker

[issue22087] asyncio: support multiprocessing (support fork)

Yury Selivanov added the comment: > The approach in Dan's patches looks sane to me and I agree the issue needs > fixing. Dan, would you like to submit a PR for this? We're using Github for > patch submissions now. Mind that the patch is very outdated and we already have these checks in

[issue22087] asyncio: support multiprocessing (support fork)

Antoine Pitrou added the comment: The approach in Dan's patches looks sane to me and I agree the issue needs fixing. Dan, would you like to submit a PR for this? We're using Github for patch submissions now. -- nosy: +pitrou stage: -> needs patch

[issue22087] asyncio: support multiprocessing (support fork)

Alexander Mohr added the comment: I believe this is now worse due to https://github.com/python/asyncio/pull/452 before I was able to simply create a new run loop from sub-processes however you will now get the error "Cannot run the event loop while another loop is running". The state of the

[issue22087] asyncio: support multiprocessing (support fork)

Changes by STINNER Victor victor.stin...@gmail.com: -- title: asyncio: support multiprocessing (support fork=) - asyncio: support multiprocessing (support fork) ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue22087

[issue22087] asyncio: support multiprocessing (support fork=)

Changes by STINNER Victor victor.stin...@gmail.com: -- title: asyncio: support multiprocessing - asyncio: support multiprocessing (support fork=) ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue22087