Skip to content

release-25.3: sql: reset schema_locked correctly in a mixed version state #149335

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.

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

Open
wants to merge 1 commit into
base: release-25.3
Choose a base branch
from

Conversation

fqazi
Copy link
Collaborator

@fqazi fqazi commented Jul 1, 2025

Backport 1/1 commits from #149271 on behalf of @fqazi.


Previously, when schema_locked was reset it would always adopt the value from create_table_with_schema_locked. This was incorrect, since this setting should only adopted if the cluster is running on 25.3. This patch adds a version gate in the reset logic.

Fixes: #149266

Release note: None


Release justification: low risk change for incorrect behavior in a mixed version state

@fqazi fqazi force-pushed the blathers/backport-release-25.3-149271 branch from bec0ff6 to 8522207 Compare July 1, 2025 14:45
@blathers-crl blathers-crl bot added blathers-backport This is a backport that Blathers created automatically. O-robot Originated from a bot. labels Jul 1, 2025
@blathers-crl blathers-crl bot requested a review from rafiss July 1, 2025 14:45
Copy link

blathers-crl bot commented Jul 1, 2025

Thanks for opening a backport.

Before merging, please confirm that the change does not break backwards compatibility and otherwise complies with the backport policy. Include a brief release justification in the PR description explaining why the backport is appropriate. All backports must be reviewed by the TL for the owning area. While the stricter LTS policy does not yet apply, please exercise judgment and consider gating non-critical changes behind a disabled-by-default feature flag when appropriate.

@blathers-crl blathers-crl bot added the backport Label PR's that are backports to older release branches label Jul 1, 2025
@cockroach-teamcity
Copy link
Member

This change is Reviewable

Previously, when schema_locked was reset it would always adopt the value
from create_table_with_schema_locked. This was incorrect, since this
setting should only adopted if the cluster is running on 25.3. This
patch adds a version gate in the reset logic.

Fixes: cockroachdb#149266

Release note: None
@fqazi fqazi force-pushed the blathers/backport-release-25.3-149271 branch from 8522207 to e4615fb Compare July 2, 2025 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Label PR's that are backports to older release branches blathers-backport This is a backport that Blathers created automatically. O-robot Originated from a bot.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants