Skip to content

Comments

Use #lease_connection if available#292

Merged
drwl merged 4 commits intodrwl:mainfrom
viralpraxis:use-lease-connection-if-available
Feb 10, 2026
Merged

Use #lease_connection if available#292
drwl merged 4 commits intodrwl:mainfrom
viralpraxis:use-lease-connection-if-available

Conversation

@viralpraxis
Copy link
Contributor

Checking out a connection via #connection should not be a problem here, but if this behavior is disallowed globally we should switch to #lease_connection.

Checking out a connection via `#connection` should not be a problem here,
but if this behavior is disallowed globally we should switch to `#lease_connection`.
@viralpraxis viralpraxis force-pushed the use-lease-connection-if-available branch from b0a360d to b38444d Compare November 13, 2025 13:15
@viralpraxis
Copy link
Contributor Author

Hey @drwl, could you take a look please?

@viralpraxis
Copy link
Contributor Author

Sorry for yet another ping @drwl, could you please take a look? 🙂

@drwl
Copy link
Owner

drwl commented Feb 10, 2026

I wasn't familiar with lease_connection so found this via Googling, adding as context: https://codewithrails.com/blog/rails-72-connection-pool-performance/

@drwl drwl merged commit 2d8d018 into drwl:main Feb 10, 2026
22 checks passed
@viralpraxis viralpraxis deleted the use-lease-connection-if-available branch February 10, 2026 22:34
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.

2 participants