Skip to content

Conversation

@rmloveland
Copy link
Contributor

Fixes:

NB. Updated description of block cache is ported to all supported versions, while update to block cache size default only applies to v25.4.

@rmloveland rmloveland requested a review from jbowens October 30, 2025 19:24
@netlify
Copy link

netlify bot commented Oct 30, 2025

Deploy Preview for cockroachdb-interactivetutorials-docs canceled.

Name Link
🔨 Latest commit ce4dd14
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-interactivetutorials-docs/deploys/691f3d04d1e7c200086935f2

@netlify
Copy link

netlify bot commented Oct 30, 2025

Deploy Preview for cockroachdb-api-docs canceled.

Name Link
🔨 Latest commit ce4dd14
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-api-docs/deploys/691f3d020272dc0008b25173

@rmloveland
Copy link
Contributor Author

@jbowens i will make sure the content that is backported does not backport the 256 MiB default value for --cache from v25.4 - otherwise I hope the rest of the content will be the same (modulo edits based on your review comments)

@netlify
Copy link

netlify bot commented Oct 30, 2025

Netlify Preview

Name Link
🔨 Latest commit ce4dd14
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-docs/deploys/691f3d027f025a0008cbad93
😎 Deploy Preview https://deploy-preview-20874--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

@jbowens jbowens left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@florence-crl florence-crl left a comment

Choose a reason for hiding this comment

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

lgtm pending non-blocking suggestions

@rmloveland
Copy link
Contributor Author

Thanks for the reviews! Backporting the cache info to all supported versions v24.1+ before merging (modulo the 256 MiB default which only applies to v25.4)

@rmloveland rmloveland force-pushed the 20251029-DOC-15012-storage-engine-block-cache branch from 9e0719f to 619ebd3 Compare November 20, 2025 15:33
@rmloveland rmloveland enabled auto-merge (squash) November 20, 2025 15:34
Fixes:

- DOC-14845
- DOC-15012

NB. Updated description of block cache is ported to all supported
versions, while update to block cache size default only applies to
v25.4.
@rmloveland rmloveland force-pushed the 20251029-DOC-15012-storage-engine-block-cache branch from 8554a33 to 1a3ed4d Compare November 20, 2025 16:07
@rmloveland rmloveland merged commit 659957a into main Nov 20, 2025
5 checks passed
@rmloveland rmloveland deleted the 20251029-DOC-15012-storage-engine-block-cache branch November 20, 2025 16:40
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