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
increase timeout (again)
  • Loading branch information
@picnixz
picnixz committed Aug 1, 2024
commit d2bfdc25b02f19b5179d9195b9266d243e4439dc
2 changes: 1 addition & 1 deletion Lib/test/test_logging.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4016,7 +4016,7 @@ def test_config_queue_handler_multiprocessing_context(self):
# use 1 process and 1 task per child to put 1 record
with ctx.Pool(1, initializer=self._mpinit_issue121723,
initargs=(q, "text"), maxtasksperchild=1):
records.append(q.get(timeout=2))
records.append(q.get(timeout=60))
self.assertTrue(q.empty())
self.assertEqual(len(records), 1)

Expand Down
Loading
0