[issue39589] Logging QueueListener should support context manager

Change by Vinay Sajip : -- resolution: -> not a bug stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue39589] Logging QueueListener should support context manager

Vinay Sajip added the comment: See my comments on the PR. I think this issue and the PR should be closed. -- ___ Python tracker ___

[issue39589] Logging QueueListener should support context manager

Change by Karthikeyan Singaravelan : -- nosy: +vinay.sajip ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue39589] Logging QueueListener should support context manager

Change by Simon : -- pull_requests: +17944 pull_request: https://github.com/python/cpython/pull/18563 ___ Python tracker ___ ___

[issue39589] Logging QueueListener should support context manager

Simon added the comment: The QueueListener in the logging library starts a background thread to monitor the log event queue. The context manager support is ideal in this case, making the code simpler, more consistent with other classes (e.g. multiprocessing.Pool) and prompts stopping the

[issue39589] Logging QueueListener should support context manager

Change by Simon : -- keywords: +patch pull_requests: +17792 stage: -> patch review pull_request: https://github.com/python/cpython/pull/18417 ___ Python tracker ___

[issue39589] Logging QueueListener should support context manager

New submission from Simon : The QueueListener could be extended to support the context manager. -- components: Library (Lib) messages: 361641 nosy: sbrugman priority: normal severity: normal status: open title: Logging QueueListener should support context manager versions: Python 3.5,