Skip to content
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.

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

Use is_broken_transaction_manager when returning pooled item #97

Merged
merged 1 commit into from
Jul 21, 2023

Conversation

dae
Copy link
Contributor

@dae dae commented Jul 21, 2023

Re: #96 (comment), is this what you had in mind?

Tentatively closes #96

@dae dae closed this Jul 21, 2023
@dae dae reopened this Jul 21, 2023
Copy link
Owner

@weiznich weiznich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 That's exactly what I believed to be already there.
I will try to cut a 0.3.1 release with that and another recent fix.

@weiznich weiznich merged commit 0045eb4 into weiznich:main Jul 21, 2023
38 of 40 checks passed
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.

Dangling transactions when pooled connections dropped
2 participants