Skip to content

Conversation

@rmloveland
Copy link
Contributor

@rmloveland rmloveland commented Jan 20, 2026

Fixes DOC-15817

@rmloveland rmloveland marked this pull request as draft January 20, 2026 16:42
@netlify
Copy link

netlify bot commented Jan 20, 2026

Deploy Preview for cockroachdb-interactivetutorials-docs canceled.

Name Link
🔨 Latest commit 4d5476f
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-interactivetutorials-docs/deploys/69729253eca52f0008753062

@netlify
Copy link

netlify bot commented Jan 20, 2026

Deploy Preview for cockroachdb-api-docs canceled.

Name Link
🔨 Latest commit 4d5476f
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-api-docs/deploys/69729253088c2a0008bfd048

@github-actions
Copy link

github-actions bot commented Jan 20, 2026

Files changed:

@netlify
Copy link

netlify bot commented Jan 20, 2026

Netlify Preview

Name Link
🔨 Latest commit 4d5476f
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-docs/deploys/69729253a561400008030331
😎 Deploy Preview https://deploy-preview-22257--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 20260120-DOC-15817-pebble-compression-analyzer branch 3 times, most recently from ee3f4ff to 9906554 Compare January 20, 2026 21:30
@rmloveland rmloveland force-pushed the 20260120-DOC-15817-pebble-compression-analyzer branch from 9906554 to 47a2e4d Compare January 20, 2026 21:31
@rmloveland rmloveland marked this pull request as ready for review January 20, 2026 21:31
@rmloveland
Copy link
Contributor Author

@RaduBerinde I think this captures the basic UX info etc from the google doc and Loom you shared. Please let me know if it is missing any important info or has inaccuracies, happy to update.

Also, q for you and @dshjoshi - I wasn't sure whether / if to say much about how best for users to approach this tool. Is the idea that they'd be working with us via Support or their assigned SEs to analyze their workloads? I would imagine most users should do this in concert with someone from CRL so they can give us feedback on possible adaptive compression designs etc? (If i read some of the internal docs right). But please let me know what you think!

Copy link
Member

@RaduBerinde RaduBerinde left a comment

Choose a reason for hiding this comment

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

Looks great.

@RaduBerinde
Copy link
Member

I wasn't sure whether / if to say much about how best for users to approach this tool. Is the idea that they'd be working with us via Support or their assigned SEs to analyze their workloads? I would imagine most users should do this in concert with someone from CRL so they can give us feedback on possible adaptive compression designs etc?

Correct. The data is generally useful to us just to make further improvements, and it can also help inform changing the compression algorithm in a specific cluster. There are other things that factor into that decision (e.g. how much CPU usage headroom there is), so ideally CRL helps with that decision.

@rmloveland
Copy link
Contributor Author

I wasn't sure whether / if to say much about how best for users to approach this tool. Is the idea that they'd be working with us via Support or their assigned SEs to analyze their workloads? I would imagine most users should do this in concert with someone from CRL so they can give us feedback on possible adaptive compression designs etc?

Correct. The data is generally useful to us just to make further improvements, and it can also help inform changing the compression algorithm in a specific cluster. There are other things that factor into that decision (e.g. how much CPU usage headroom there is), so ideally CRL helps with that decision.

ok thanks Radu! i guess this is somewhat covered under the 'Use cases' section where we say

This data can help Cockroach Labs evaluate compression defaults and can help you evaluate whether an alternate compression algorithm is appropriate for your workload.

so for now I'll leave it at that, but we can update it as needed with more words if we encounter situations where customers need clarification

thank you for the review!

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 suggestion and question.

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.

additional suggestions

@rmloveland
Copy link
Contributor Author

thanks for the reviews! merging now ...

@rmloveland rmloveland enabled auto-merge (squash) January 22, 2026 21:10
@rmloveland rmloveland merged commit 927b971 into main Jan 22, 2026
7 checks passed
@rmloveland rmloveland deleted the 20260120-DOC-15817-pebble-compression-analyzer branch January 22, 2026 21:27
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