Skip to content

Conversation

rmloveland
Copy link
Contributor

Fixes:

Summary of changes:

  • This update removes all mentions of the default value in prose, and instead points the user to the single source of truth on the cluster settings page.

  • Files updated: READ COMMITTED page, and an include used on the 'Transactions' and 'Transaction Retry Error Reference' pages

Fixes:

- DOC-10360
- DOC-12305

Summary of changes:

- This update removes all mentions of the default value in prose, and
  instead points the user to the single source of truth on the cluster
  settings page.

- Files updated: `READ COMMITTED` page, and an include used on the
  'Transactions' and 'Transaction Retry Error Reference' pages
@rmloveland rmloveland requested review from rafiss and smcvey October 8, 2025 19:20
Copy link

netlify bot commented Oct 8, 2025

Deploy Preview for cockroachdb-interactivetutorials-docs canceled.

Name Link
🔨 Latest commit a3b15a7
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-interactivetutorials-docs/deploys/68e80bb654ab5c000814d5ca

Copy link

netlify bot commented Oct 8, 2025

Deploy Preview for cockroachdb-api-docs canceled.

Name Link
🔨 Latest commit a3b15a7
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-api-docs/deploys/68e80bb66a4e060008cd77af

Copy link

github-actions bot commented Oct 8, 2025

@rmloveland
Copy link
Contributor Author

@smcvey adding you FYI so you know this is getting done, thanks again for filing the issue!

@rafiss looking at cockroachdb/cockroach#124580 that PR had the v24.1 backport label. Is it safe to assume this docs change (once approved for technical correctness) can be ported to all versions of the docs v24.1+ ?

Copy link

netlify bot commented Oct 8, 2025

Netlify Preview

Name Link
🔨 Latest commit a3b15a7
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-docs/deploys/68e80bb66ce29000080b919a
😎 Deploy Preview https://deploy-preview-20516--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.

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.

lgtm!

@rmloveland rmloveland enabled auto-merge (squash) October 9, 2025 19:25
@rmloveland rmloveland merged commit 7d48082 into main Oct 9, 2025
6 checks passed
@rmloveland rmloveland deleted the 20251008-DOC-12305-update-sql-defaults-results-buffer-size branch October 9, 2025 19:53
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