diff options
author | Yury Selivanov <yury@magic.io> | 2018-05-30 00:56:33 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-05-30 00:56:33 (GMT) |
commit | af9cda9845666e2f704177a431d29f91efbf828a (patch) | |
tree | ecd1b22e6e1a7dad91c5065bfb71ae9dec2bbef5 | |
parent | 3ddee64587482082fcbab930ff82ed67aa9f4a0d (diff) | |
download | cpython-af9cda9845666e2f704177a431d29f91efbf828a.zip cpython-af9cda9845666e2f704177a431d29f91efbf828a.tar.gz cpython-af9cda9845666e2f704177a431d29f91efbf828a.tar.bz2 |
Revert "bpo-22087: Fix Policy.get_event_loop() to detect fork (GH-7208)" (GH-7233)
This reverts commit 2a7eb0b531656f4a77d85078e6e009e4b3639ef9.
-rw-r--r-- | Lib/asyncio/events.py | 7 | ||||
-rw-r--r-- | Lib/test/test_asyncio/test_unix_events.py | 32 | ||||
-rw-r--r-- | Misc/NEWS.d/next/Library/2018-05-29-12-06-54.bpo-22087.uv7_Y6.rst | 3 |
3 files changed, 0 insertions, 42 deletions
diff --git a/Lib/asyncio/events.py b/Lib/asyncio/events.py index 68dc25e..40946bb 100644 --- a/Lib/asyncio/events.py +++ b/Lib/asyncio/events.py @@ -625,23 +625,16 @@ class BaseDefaultEventLoopPolicy(AbstractEventLoopPolicy): class _Local(threading.local): _loop = None - _pid = None _set_called = False def __init__(self): self._local = self._Local() - self._local._pid = os.getpid() def get_event_loop(self): """Get the event loop. This may be None or an instance of EventLoop. """ - if self._local._pid != os.getpid(): - # If we detect we're in a child process forked by multiprocessing, - # we reset self._local so that we'll get a new event loop. - self._local = self._Local() - if (self._local._loop is None and not self._local._set_called and isinstance(threading.current_thread(), threading._MainThread)): diff --git a/Lib/test/test_asyncio/test_unix_events.py b/Lib/test/test_asyncio/test_unix_events.py index 5775668..104f995 100644 --- a/Lib/test/test_asyncio/test_unix_events.py +++ b/Lib/test/test_asyncio/test_unix_events.py @@ -13,7 +13,6 @@ import sys import tempfile import threading import unittest -import multiprocessing from unittest import mock from test import support @@ -1793,37 +1792,6 @@ class FastChildWatcherTests (ChildWatcherTestsMixin, test_utils.TestCase): return asyncio.FastChildWatcher() -class ForkedProcessTests(unittest.TestCase): - def setUp(self): - self.parent_loop = asyncio.SelectorEventLoop() - asyncio.set_event_loop(self.parent_loop) - self.ctx = multiprocessing.get_context("fork") - - def tearDown(self): - self.parent_loop.close() - - def _check_loops_not_equal(self, old_loop): - loop = asyncio.get_event_loop() - if loop is old_loop: - raise RuntimeError("Child process inherited parent's event loop") - - try: - val = loop.run_until_complete(asyncio.sleep(0.05, result=42)) - if val != 42: - raise RuntimeError("new event loop does not work") - finally: - loop.close() - - sys.exit(loop is old_loop) - - def test_new_loop_in_child(self): - p = self.ctx.Process(target=self._check_loops_not_equal, - args=(self.parent_loop,)) - p.start() - p.join() - self.assertEqual(p.exitcode, 0) - - class PolicyTests(unittest.TestCase): def create_policy(self): diff --git a/Misc/NEWS.d/next/Library/2018-05-29-12-06-54.bpo-22087.uv7_Y6.rst b/Misc/NEWS.d/next/Library/2018-05-29-12-06-54.bpo-22087.uv7_Y6.rst deleted file mode 100644 index 92127cd..0000000 --- a/Misc/NEWS.d/next/Library/2018-05-29-12-06-54.bpo-22087.uv7_Y6.rst +++ /dev/null @@ -1,3 +0,0 @@ -Fix Policy.get_event_loop() to detect fork and return a new loop. - -Original patch by Dan O'Reilly. |