Skip to content

Conversation

@rmloveland
Copy link
Contributor

@rmloveland rmloveland commented Jul 29, 2025

Fixes:

Summary of changes:

  • Add create_table_with_schema_locked session var docs, with a link to
    the schema_locked table storage parameter.

  • Since table storage parameters can be added during CREATE TABLE, and
    the list of storage params is in an include file, we share the include
    file into the CREATE TABLE docs as well (they were already in ALTER TABLE docs).

@netlify
Copy link

netlify bot commented Jul 29, 2025

Deploy Preview for cockroachdb-api-docs canceled.

Name Link
🔨 Latest commit 587dbfe
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-api-docs/deploys/688a345dbc256600078c4cc8

@netlify
Copy link

netlify bot commented Jul 29, 2025

Deploy Preview for cockroachdb-interactivetutorials-docs canceled.

Name Link
🔨 Latest commit 587dbfe
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-interactivetutorials-docs/deploys/688a345d4df1e40008114b65

@github-actions
Copy link

github-actions bot commented Jul 29, 2025

Files changed:

@netlify
Copy link

netlify bot commented Jul 29, 2025

Netlify Preview

Name Link
🔨 Latest commit 587dbfe
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-docs/deploys/688a345d00943c0008ef2d8c
😎 Deploy Preview https://deploy-preview-20013--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 20250729-DOC-11544-schema_locked branch from ba13a60 to 1c66473 Compare July 29, 2025 17:23
Fixes DOC-11544

Summary of changes:

- Add `create_table_with_schema_locked` session var docs, with a link to
  the `schema_locked` table storage parameter.

- Since table storage parameters can be added during `CREATE TABLE`, and
  the list of storage params is in an include file, we share the include
  file into the `CREATE TABLE` docs as well (they were already in `ALTER
  TABLE` docs).
@rmloveland rmloveland force-pushed the 20250729-DOC-11544-schema_locked branch from 1c66473 to 491155c Compare July 29, 2025 18:14
@rmloveland rmloveland marked this pull request as ready for review July 29, 2025 18:14
@rmloveland rmloveland requested a review from fqazi July 29, 2025 18:14
@rmloveland rmloveland requested a review from taroface July 29, 2025 20:25
Copy link
Contributor

@taroface taroface left a comment

Choose a reason for hiding this comment

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

LGTM, one optional suggestion

@rmloveland rmloveland enabled auto-merge (squash) July 30, 2025 14:49
@rmloveland rmloveland merged commit f87dd09 into main Jul 30, 2025
5 checks passed
@rmloveland rmloveland deleted the 20250729-DOC-11544-schema_locked branch July 30, 2025 15:10
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