Open mattlord opened 17 hours ago
Hello reviewers! :wave: Please follow this checklist when reviewing this Pull Request.
release notes (needs details)
label if users need to know about this change.-
), and have a clear help text.Jobs
should be named in order to mark it as required
.required
, the maintainer team must be notified._vt
tables and RPCs need to be backward compatible.vtctl
command output order should be stable and awk
-able.Attention: Patch coverage is 75.00000%
with 5 lines
in your changes missing coverage. Please review.
Project coverage is 69.44%. Comparing base (
2e2b223
) to head (ab83107
). Report is 4 commits behind head on main.
Files with missing lines | Patch % | Lines |
---|---|---|
go/vt/vtctl/workflow/traffic_switcher.go | 75.00% | 5 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description
You can specify the columns which make up a vindex in two ways within the vschema:
The
MoveTables
target sequence initialization work (added in https://github.com/vitessio/vitess/pull/13656) only supports the legacy way. It needs to be updated to support either.That's exactly what this PR does. 🙂
I think that we should backport this to v18 because:
Related Issue(s)
Checklist