Skip to content

Conversation

@vladvildanov
Copy link
Contributor

Fixes #1506

@coveralls
Copy link

coveralls commented Feb 20, 2025

Coverage Status

coverage: 92.766% (+0.008%) from 92.758%
when pulling be6cef0 on vv-multiple-persistent-conn-fix
into 974379b on main.

@tillkruss
Copy link
Member

Should we move this into v2?

@vladvildanov
Copy link
Contributor Author

@tillkruss I think we should backport it everywhere. So let's merge it into main and I will backport it into 2.x and 1.x (if it's still open for critical bugs)

@tillkruss
Copy link
Member

v1.x is frozen, let's just merge into v2.x as well.

@tillkruss tillkruss changed the base branch from main to v2.x February 21, 2025 17:25
@tillkruss tillkruss requested a review from a team February 21, 2025 17:25
@tillkruss tillkruss changed the base branch from v2.x to main February 21, 2025 17:25
@vladvildanov vladvildanov merged commit 09e88b6 into main Feb 24, 2025
50 checks passed
@vladvildanov vladvildanov deleted the vv-multiple-persistent-conn-fix branch February 24, 2025 07:51
vladvildanov added a commit that referenced this pull request Feb 25, 2025
@vladvildanov vladvildanov mentioned this pull request Feb 25, 2025
tillkruss pushed a commit that referenced this pull request Feb 25, 2025
* Backport #1512
* Removed outdated test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

4 participants