Skip to content

Conversation

@tanjinx
Copy link

@tanjinx tanjinx commented Aug 5, 2025

Adds a new vtgate flag --default-multi-shard-autocommit which, as the name implies, opts the query engine into using multi-shard autocommit semantics by default, even if the plan does not contain the query directive MULTI_SHARD_AUTOCOMMIT.

Description

backport upstream vitessio#17635

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

@github-actions github-actions bot added this to the v19.0.7 milestone Aug 5, 2025
Adds a new vtgate flag --default-multi-shard-autocommit which, as the name
implies, opts the query engine into using multi-shard autocommit semantics by
default, even if the plan does not contain the query directive
MULTI_SHARD_AUTOCOMMIT.

Signed-off-by: Michael Demmer <[email protected]>
Signed-off-by: Tanjin Xu <[email protected]>
@tanjinx tanjinx force-pushed the backport-multi-shard-auto-commit-default branch from 7815dbf to fcf0b9c Compare August 5, 2025 17:54
Signed-off-by: Tanjin Xu <[email protected]>
@tanjinx tanjinx requested review from demmer and venkatraju August 5, 2025 19:07
tanjinx added 2 commits August 5, 2025 12:14
Signed-off-by: Tanjin Xu <[email protected]>
Signed-off-by: Tanjin Xu <[email protected]>
@tanjinx tanjinx marked this pull request as ready for review August 5, 2025 22:56
@tanjinx tanjinx requested a review from a team as a code owner August 5, 2025 22:56
@tanjinx tanjinx modified the milestone: v19.0.7 Aug 5, 2025
@tanjinx tanjinx merged commit e4761bb into slack-19.0 Aug 5, 2025
163 of 167 checks passed
@tanjinx tanjinx deleted the backport-multi-shard-auto-commit-default branch August 5, 2025 23:26
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.

4 participants