8000 gh-121723: Relax constraints on queue objects for `logging.handlers.QueueHandler`. by picnixz · Pull Request #122154 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

gh-121723: Relax constraints on queue objects for logging.handlers.QueueHandler. #122154

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
merged 15 commits into from
Aug 2, 2024
Merged

gh-121723: Relax constraints on queue objects for logging.handlers.QueueHandler. #122154

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
6ab2257
Relax constraints on queue objects for queue handlers.
picnixz Jul 23, 2024
9af323b
add tests
picnixz Jul 23, 2024
78936a1
update docs
picnixz Jul 23, 2024
5fc3d5a
blurb
picnixz Jul 23, 2024
04c7813
refactor into a helper
picnixz Jul 24, 2024
e5cdb1c
add regression test
picnixz Jul 24, 2024
b770787
fix start methods selection on MS-Windows
picnixz Jul 24, 2024
65a8987
fixup refs
picnixz Jul 24, 2024
7729fb6
increase the timeout (maybe it works like this?)
picnixz Jul 24, 2024
57d7190
fixup! typos
picnixz Jul 28, 2024
e4800c4
fixup! typos
picnixz Jul 28, 2024
bd12893
reduce line numbers
picnixz Jul 28, 2024
70d69e5
renaming
picnixz Jul 28, 2024
d6b6ef3
fixup! refs
picnixz Jul 29, 2024
d2bfdc2
increase timeout (again)
picnixz Aug 1, 2024
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
fix start methods selection on MS-Windows
  • Loading branch information
@picnixz
picnixz committed Jul 24, 2024
commit b770787e1f634b4fff39f77954e3708aa2c7e7df
10 changes: 7 additions & 3 deletions Lib/test/test_logging.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4002,9 +4002,13 @@ def test_config_queue_handler_invalid_config_does_not_create_multiprocessing_man
@unittest.skipUnless(support.Py_DEBUG, "requires a debug build")
def test_config_queue_handler_multiprocessing_context(self):
# regression test for gh-121723
for startmethod in ['fork', 'spawn', 'forkserver']:
with self.subTest(startmethod=startmethod):
ctx = multiprocessing.get_context(startmethod)
if support.MS_WINDOWS:
start_methods = ['spawn']
else:
start_methods = ['spawn', 'fork', 'forkserver']
for start_method in start_methods:
with self.subTest(start_method=start_method):
ctx = multiprocessing.get_context(start_method)
with ctx.Manager() as manager:
q = manager.Queue()
records = []
Expand Down
Loading
0