Skip to content

Update schema changes docs with schema_locked info#21726

Merged
rmloveland merged 2 commits intomainfrom
20251212-DOC-15240-schema_locked-etc
Feb 10, 2026
Merged

Update schema changes docs with schema_locked info#21726
rmloveland merged 2 commits intomainfrom
20251212-DOC-15240-schema_locked-etc

Conversation

@rmloveland
Copy link
Contributor

Fixes DOC-15240

@rmloveland rmloveland marked this pull request as draft December 18, 2025 16:40
@netlify
Copy link

netlify bot commented Dec 18, 2025

Deploy Preview for cockroachdb-interactivetutorials-docs canceled.

Name Link
🔨 Latest commit 6608f7a
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-interactivetutorials-docs/deploys/698b8229b599cb000859bda7

@netlify
Copy link

netlify bot commented Dec 18, 2025

Deploy Preview for cockroachdb-api-docs canceled.

Name Link
🔨 Latest commit 6608f7a
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-api-docs/deploys/698b8229771db60008baa148

@github-actions
Copy link

github-actions bot commented Dec 18, 2025

@netlify
Copy link

netlify bot commented Dec 18, 2025

Netlify Preview

Name Link
🔨 Latest commit 6608f7a
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-docs/deploys/698b8229c7e72e0008f62e04
😎 Deploy Preview https://deploy-preview-21726--cockroachdb-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@rmloveland rmloveland force-pushed the 20251212-DOC-15240-schema_locked-etc branch 3 times, most recently from a2eb098 to 2968c4e Compare February 2, 2026 19:56
@rmloveland rmloveland marked this pull request as ready for review February 2, 2026 19:56
@rmloveland
Copy link
Contributor Author

@rmloveland TODO: backport these changes (minus the mention of the v26.1 defaults) to supported versions v24.1 and later (pending technical reviewer confirmation)

Copy link
Contributor

@rafiss rafiss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks! the extra linkage is helpful, but we still need to find an accurate way and helpful way to describe the setting. unfortunately, the name schema_locked is quite misleading

@rmloveland rmloveland requested a review from rafiss February 4, 2026 22:00
Copy link
Contributor

@rafiss rafiss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the new phrasing lgtm!

Copy link
Contributor

@bsanchez-the-roach bsanchez-the-roach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One non-blocking nit, otherwise LGTM

Fixes DOC-15240

NB. These changes are applied to v25.4 and v26.1
@rmloveland rmloveland force-pushed the 20251212-DOC-15240-schema_locked-etc branch from 66bf7d7 to dd116b4 Compare February 10, 2026 19:04
@rmloveland
Copy link
Contributor Author

thanks for the reviews! backported to v25.4 (modulo default description) and merging ...

@rmloveland rmloveland enabled auto-merge (squash) February 10, 2026 19:08
@rmloveland rmloveland merged commit c395109 into main Feb 10, 2026
7 checks passed
@rmloveland rmloveland deleted the 20251212-DOC-15240-schema_locked-etc branch February 10, 2026 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants