[issue37470] Make it explicit what happens when using a bounded queue with QueueHandler
New submission from Martijn Pieters <mj@python.org>: The documentation doesn't make it explicit what happens if you use a bounded queue together with logging.handlers.QueueHandler. If the queue is bounded in size and attempts are made to add logrecords faster than a queue listener removes them, then the resulting `queue.Full` exception is passed to `handler.handleError()` and that usually means the record is simply dropped (see https://docs.python.org/3/library/logging.html#logging.Handler.handleError). That may be the desired behaviour, but making it explicit is always better. ---------- assignee: docs@python components: Documentation messages: 347018 nosy: docs@python, mjpieters priority: normal severity: normal status: open title: Make it explicit what happens when using a bounded queue with QueueHandler versions: Python 3.7, Python 3.8, Python 3.9 _______________________________________ Python tracker <report@bugs.python.org> <https://bugs.python.org/issue37470> _______________________________________
Change by Karthikeyan Singaravelan <tir.karthi@gmail.com>: ---------- nosy: +vinay.sajip _______________________________________ Python tracker <report@bugs.python.org> <https://bugs.python.org/issue37470> _______________________________________
Change by Vinay Sajip <vinay_sajip@yahoo.co.uk>: ---------- keywords: +patch pull_requests: +14345 stage: -> patch review pull_request: https://github.com/python/cpython/pull/14532 _______________________________________ Python tracker <report@bugs.python.org> <https://bugs.python.org/issue37470> _______________________________________
Vinay Sajip <vinay_sajip@yahoo.co.uk> added the comment: New changeset 0f4e8132820947d93eccf31b9e526b81c6ffa53d by Vinay Sajip in branch 'master': bpo-37470: Document more clearly the error handling for QueueHandler.emit(). (GH-14532) https://github.com/python/cpython/commit/0f4e8132820947d93eccf31b9e526b81c6f... ---------- _______________________________________ Python tracker <report@bugs.python.org> <https://bugs.python.org/issue37470> _______________________________________
Change by miss-islington <mariatta.wijaya+miss-islington@gmail.com>: ---------- pull_requests: +14346 pull_request: https://github.com/python/cpython/pull/14533 _______________________________________ Python tracker <report@bugs.python.org> <https://bugs.python.org/issue37470> _______________________________________
Change by miss-islington <mariatta.wijaya+miss-islington@gmail.com>: ---------- pull_requests: +14347 pull_request: https://github.com/python/cpython/pull/14534 _______________________________________ Python tracker <report@bugs.python.org> <https://bugs.python.org/issue37470> _______________________________________
Vinay Sajip <vinay_sajip@yahoo.co.uk> added the comment: New changeset 844a9d64a4f640d1b20dc6ea54ab375680332d93 by Vinay Sajip (Miss Islington (bot)) in branch '3.7': bpo-37470: Document more clearly the error handling for QueueHandler.emit(). (GH-14532) (GH-14534) https://github.com/python/cpython/commit/844a9d64a4f640d1b20dc6ea54ab3756803... ---------- _______________________________________ Python tracker <report@bugs.python.org> <https://bugs.python.org/issue37470> _______________________________________
Vinay Sajip <vinay_sajip@yahoo.co.uk> added the comment: New changeset 91f9f098fcdb023dbb89d06c8833e89a11cbae4c by Vinay Sajip (Miss Islington (bot)) in branch '3.8': bpo-37470: Document more clearly the error handling for QueueHandler.emit(). (GH-14532) (GH-14533) https://github.com/python/cpython/commit/91f9f098fcdb023dbb89d06c8833e89a11c... ---------- _______________________________________ Python tracker <report@bugs.python.org> <https://bugs.python.org/issue37470> _______________________________________
Change by Vinay Sajip <vinay_sajip@yahoo.co.uk>: ---------- resolution: -> fixed stage: patch review -> resolved status: open -> closed _______________________________________ Python tracker <report@bugs.python.org> <https://bugs.python.org/issue37470> _______________________________________
participants (4)
-
Karthikeyan Singaravelan -
Martijn Pieters
-
miss-islington
-
Vinay Sajip