Skip to content

[release-23.0] vtgate: fix handling of session variables on targeted connections (#19318)#19335

Merged
arthurschreiber merged 2 commits into
release-23.0from
backport-19318-to-release-23.0
Feb 9, 2026
Merged

[release-23.0] vtgate: fix handling of session variables on targeted connections (#19318)#19335
arthurschreiber merged 2 commits into
release-23.0from
backport-19318-to-release-23.0

Conversation

@vitess-bot
Copy link
Copy Markdown
Contributor

@vitess-bot vitess-bot Bot commented Feb 9, 2026

Description

This is a backport of #19318

@vitess-bot
Copy link
Copy Markdown
Contributor Author

vitess-bot Bot commented Feb 9, 2026

Hello @arthurschreiber, there are conflicts in this backport.

Please address them in order to merge this Pull Request. You can execute the snippet below to reset your branch and resolve the conflict manually.

Make sure you replace origin by the name of the vitessio/vitess remote

git fetch --all
gh pr checkout 19335
git reset --hard origin/release-23.0
git cherry-pick -m 1 0b345d26c8d2eea480291514de962e94dbc2e005

@github-actions github-actions Bot added this to the v23.0.2 milestone Feb 9, 2026
@arthurschreiber arthurschreiber removed Skip CI Skip CI actions from running Merge Conflict labels Feb 9, 2026
Signed-off-by: Arthur Schreiber <arthur@planetscale.com>
@arthurschreiber arthurschreiber force-pushed the backport-19318-to-release-23.0 branch from b6929b8 to 33a7ba3 Compare February 9, 2026 16:04
@arthurschreiber arthurschreiber marked this pull request as ready for review February 9, 2026 16:06
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.72%. Comparing base (7c76856) to head (33a7ba3).
⚠️ Report is 3 commits behind head on release-23.0.

Additional details and impacted files
@@               Coverage Diff                @@
##           release-23.0   #19335      +/-   ##
================================================
- Coverage         69.73%   69.72%   -0.02%     
================================================
  Files              1606     1606              
  Lines            214823   214818       -5     
================================================
- Hits             149798   149772      -26     
- Misses            65025    65046      +21     

☔ 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.

@arthurschreiber arthurschreiber merged commit dff9f68 into release-23.0 Feb 9, 2026
170 of 176 checks passed
@arthurschreiber arthurschreiber deleted the backport-19318-to-release-23.0 branch February 9, 2026 19:14
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.72%. Comparing base (7c76856) to head (33a7ba3).
⚠️ Report is 4 commits behind head on release-23.0.

Additional details and impacted files
@@               Coverage Diff                @@
##           release-23.0   #19335      +/-   ##
================================================
- Coverage         69.73%   69.72%   -0.02%     
================================================
  Files              1606     1606              
  Lines            214823   214818       -5     
================================================
- Hits             149798   149772      -26     
- Misses            65025    65046      +21     

☔ 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants