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.

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

Jump to bottom

Empty connection pool error handling #41

Merged
merged 3 commits into from
Sep 12, 2024
Merged

Empty connection pool error handling #41

merged 3 commits into from
Sep 12, 2024

Conversation

noodl
Copy link
Contributor

@noodl noodl commented Apr 25, 2023

Fixes an issue where nil is added to the connection pool when check_out raises an exception (such as when the pool is exhausted) and an attempt is made to run a query.

@noodl noodl added the bug Something isn't working label Apr 25, 2023
@noodl noodl self-assigned this Apr 25, 2023
@noodl noodl changed the title Connection pool empty error handling Empty connection pool error handling Apr 25, 2023
@noodl noodl merged commit 1d98c86 into master Sep 12, 2024
@noodl noodl deleted the connection-pool-full-error-handling branch September 12, 2024 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adamgeorgeson adamgeorgeson adamgeorgeson approved these changes

Assignees

@noodl noodl

Labels
bug Something isn't working
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@noodl @adamgeorgeson