Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

sql: fix LSC that incorrectly missed secondary index rewrite during ALTER PK #114622

Merged
merged 1 commit into from
Nov 21, 2023
Merged

sql: fix LSC that incorrectly missed secondary index rewrite during ALTER PK #114622

merged 1 commit into from
Nov 21, 2023

Commits on Nov 20, 2023

  1. sql: fix LSC that incorrectly missed secondary index rewrite during A… 

    …LTER PK

Fixes cockroachdb#114436

Release note (bug fix): Previously, when set `use_declarative_schema_changer=off`
and we attempt to alter primary key on a table that has unique secondary
indexes on new-pk-cols, the unique secondary index would incorrectly
still have old-pk-cols as its keySuffixColumn after the ALTER PK. This
is problematic because a subsequent dropping of the old-pk-cols would
unexpectedly drop those unique secondary indexes as well, even without
CASCADE. This commit fixes that.
    @Xiang-Gu
    Xiang-Gu committed Nov 20, 2023
    Configuration menu
    Copy the full SHA
    8f293f6 View commit details
    Browse the repository at this point in the history