8000 gh-117531: Unblock getters after non-immediate queue shutdown by EpicWink · Pull Request #117532 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

gh-117531: Unblock getters after non-immediate queue shutdown #117532

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 10 commits into from
Apr 10, 2024
Merged

gh-117531: Unblock getters after non-immediate queue shutdown #117532

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
5d490b3
Unblock getters after non-immediate queue shutdown
EpicWink Apr 4, 2024
076122d
Increase join timeout in test
EpicWink Apr 4, 2024
835d45e
Merge remote-tracking branch 'upstream/main' into queue-shutdown-unbl…
EpicWink Apr 4, 2024
5c709bf
Improve test name
EpicWink Apr 4, 2024
f935277
Explicit non-immediate shutdown in test
EpicWink Apr 4, 2024
14caf34
Merge remote-tracking branch 'upstream/main' into queue-shutdown-unbl…
EpicWink Apr 8, 2024
61d2010
Mention gets are always unblocked
EpicWink Apr 8, 2024
a67af7a
Comment reason for waking-up getters after shutdown
EpicWink Apr 8, 2024
5d90998
Merge remote-tracking branch 'upstream/main' into queue-shutdown-unbl…
EpicWink Apr 9, 2024
dbb2190
Move comment to correct position
EpicWink Apr 9, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loa 8000 ding
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Move comment to correct position
  • Loading branch information
@EpicWink
EpicWink committed Apr 9, 2024
commit dbb2190b7477cef939c21aff2de91850d3dc3a2a
2 changes: 1 addition & 1 deletion Lib/queue.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -252,8 +252,8 @@ def shutdown(self, immediate=False):
self.unfinished_tasks -= 1
# release all blocked threads in `join()`
self.all_tasks_done.notify_all()
self.not_empty.notify_all()
# All getters need to re-check queue-empty to raise ShutDown
self.not_empty.notify_all()
self.not_full.notify_all()

# Override these methods to implement other queue organizations
Expand Down
0