Skip to content

Pool.close: close all connections before returning#3952

Merged
abonander merged 3 commits intolaunchbadge:mainfrom
jpmelos:pool-close-race-condition
Aug 16, 2025
Merged

Pool.close: close all connections before returning#3952
abonander merged 3 commits intolaunchbadge:mainfrom
jpmelos:pool-close-race-condition

Conversation

@jpmelos
Copy link
Copy Markdown
Contributor

@jpmelos jpmelos commented Jul 29, 2025

Fixes #3217.
Supersedes #3299.

I'm taking into consideration the discussion that already happened in #3299, and then I just rebase it on the latest main and add one commit on top of that to address the problem with child pools. The solution turned out to be pretty simple.

@abonander abonander merged commit 5f8fc6b into launchbadge:main Aug 16, 2025
92 checks passed
@jpmelos jpmelos deleted the pool-close-race-condition branch August 16, 2025 02:26
@pronebird
Copy link
Copy Markdown

@abonander this is a very critical patch. I can't wait to upgrade. Any idea when this is going to be released? Thanks

@pronebird
Copy link
Copy Markdown

@abonander it's been 4 months. Is there any plan to release a patch?

@dzmitry-lahoda

This comment was marked as spam.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pool::close does not always wait for all connections to close

5 participants