[proxy] github.com← back | site home | direct (HTTPS) ↗ | proxy home | ◑ dark◐ light

bpo-36813: Fix QueueListener to call task_done() upon termination. by bharel · Pull Request #13113 · python/cpython

bharel

added 2 commits

May 6, 2019 17:44

Merge

update base

…into fix-queuelistener

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request

Jun 1, 2019
…ythonGH-13113)

Fixed QueueListener in order to avoid random deadlocks.
Unable to add regression tests atm due to time constraints, will add it in a bit.
Regarding implementation, although it's nested, it does not cause performance issues whatsoever, and does not call task_done() in case of an exception (which is the right thing to do IMHO).

https://bugs.python.org/issue36813
(cherry picked from commit 6b282e1)

Co-authored-by: Bar Harel <bzvi7919@gmail.com>

miss-islington added a commit that referenced this pull request

Jun 1, 2019
…H-13113)

Fixed QueueListener in order to avoid random deadlocks.
Unable to add regression tests atm due to time constraints, will add it in a bit.
Regarding implementation, although it's nested, it does not cause performance issues whatsoever, and does not call task_done() in case of an exception (which is the right thing to do IMHO).

https://bugs.python.org/issue36813
(cherry picked from commit 6b282e1)

Co-authored-by: Bar Harel <bzvi7919@gmail.com>

bharel deleted the fix-queuelistener branch

December 23, 2019 12:37

DinoV pushed a commit to DinoV/cpython that referenced this pull request

Jan 14, 2020
…ythonGH-13113)

Fixed QueueListener in order to avoid random deadlocks.
Unable to add regression tests atm due to time constraints, will add it in a bit.
Regarding implementation, although it's nested, it does not cause performance issues whatsoever, and does not call task_done() in case of an exception (which is the right thing to do IMHO).


https://bugs.python.org/issue36813

Closed