Skip to content

Comments

[slack-22.0]: backport 18967: connpool: fix connection leak during idle connection reopen#758

Merged
tanjinx merged 1 commit intoslack-22.0from
backport-18967-v22
Dec 4, 2025
Merged

[slack-22.0]: backport 18967: connpool: fix connection leak during idle connection reopen#758
tanjinx merged 1 commit intoslack-22.0from
backport-18967-v22

Conversation

@tanjinx
Copy link

@tanjinx tanjinx commented Dec 2, 2025

…eopen (vitessio#18967) (vitessio#18970)

Description

backport vitessio#18970 (vitessio#18967)

Related Issue(s)

Checklist

  • "Backport to:" labels have been added if this change should be back-ported to release branches
  • If this change is to be back-ported to previous releases, a justification is included in the PR description
  • Tests were added or are not required
  • Did the new or modified tests pass consistently locally and on CI?
  • Documentation was added or is not required

Deployment Notes

AI Disclosure

…eopen (vitessio#18967) (vitessio#18970)

Signed-off-by: Arthur Schreiber <arthur@planetscale.com>
Co-authored-by: Arthur Schreiber <arthurschreiber@github.com>
@tanjinx tanjinx requested a review from a team as a code owner December 2, 2025 18:11
@github-actions github-actions bot added this to the v22.0.1 milestone Dec 2, 2025
@tanjinx tanjinx changed the title [slack-22.0]: backport 18967 [release-22.0] connpool: fix connection leak during idle connection r… [slack-22.0]: backport 18967: connpool: fix connection leak during idle connection r… Dec 2, 2025
@tanjinx tanjinx changed the title [slack-22.0]: backport 18967: connpool: fix connection leak during idle connection r… [slack-22.0]: backport 18967: connpool: fix connection leak during idle connection reopen Dec 2, 2025
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.72%. Comparing base (b1ab347) to head (c184fae).
⚠️ Report is 20 commits behind head on slack-22.0.

Additional details and impacted files
@@              Coverage Diff               @@
##           slack-22.0     #758      +/-   ##
==============================================
+ Coverage       67.53%   69.72%   +2.19%     
==============================================
  Files            1600     1604       +4     
  Lines          261782   213319   -48463     
==============================================
- Hits           176786   148747   -28039     
+ Misses          84996    64572   -20424     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@tanjinx tanjinx merged commit ae321b8 into slack-22.0 Dec 4, 2025
185 of 190 checks passed
@tanjinx tanjinx deleted the backport-18967-v22 branch December 4, 2025 20:06
sbaker617 pushed a commit that referenced this pull request Feb 5, 2026
…eopen (vitessio#18967) (vitessio#18970) (#758)

Signed-off-by: Arthur Schreiber <arthur@planetscale.com>
Co-authored-by: vitess-bot[bot] <108069721+vitess-bot[bot]@users.noreply.github.com>
Co-authored-by: Arthur Schreiber <arthurschreiber@github.com>
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.

3 participants