Skip to content

Add benchmark link and NVMe footnote to Managed Postgres overview#6287

Merged
yashpreetbathla merged 4 commits into
mainfrom
yashpreet/managed-postgres-nvme-benchmark-links
May 27, 2026
Merged

Add benchmark link and NVMe footnote to Managed Postgres overview#6287
yashpreetbathla merged 4 commits into
mainfrom
yashpreet/managed-postgres-nvme-benchmark-links

Conversation

@yashpreetbathla
Copy link
Copy Markdown
Contributor

@yashpreetbathla yashpreetbathla commented May 26, 2026

Summary

  • Added a superscript footnote [*] on "Unlimited local IOPS" linking to the AWS EC2 instance store documentation
  • Added a link to the Performance benchmarks page for readers who want detailed benchmark results
  • Added a standalone line linking to AWS local NVMe limits for more details
image

Checklist


Note

Low Risk
Documentation-only edits with no runtime, security, or data-path impact.

Overview
The Managed Postgres overview doc now qualifies the “Unlimited local IOPS” bullet with a footnote marker and adds reader-facing pointers instead of leaving that claim standalone.

It links to the internal Performance benchmarks page for detailed numbers, and adds a short line with AWS EC2 instance store documentation (memory-, storage-, and CPU-optimized families) so “unlimited” is understood in terms of local NVMe limits on AWS.

Reviewed by Cursor Bugbot for commit d4f5954. Bugbot is set up for automated code reviews on this repo. Configure here.

@yashpreetbathla yashpreetbathla requested a review from a team as a code owner May 26, 2026 22:54
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 26, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ Blargian
❌ yashpreetbathla
You have signed the CLA already but the status is still pending? Let us recheck it.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 26, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
clickhouse-docs Ready Ready Preview, Comment May 27, 2026 5:54am
4 Skipped Deployments
Project Deployment Actions Updated (UTC)
clickhouse-docs-jp Ignored Ignored May 27, 2026 5:54am
clickhouse-docs-ko Ignored Ignored Preview May 27, 2026 5:54am
clickhouse-docs-ru Ignored Ignored Preview May 27, 2026 5:54am
clickhouse-docs-zh Ignored Ignored Preview May 27, 2026 5:54am

Request Review

Comment thread docs/cloud/managed-postgres/overview.md Outdated

- **Microsecond-level disk latency** instead of milliseconds
- **Unlimited local IOPS** without network bottlenecks
- **Unlimited local IOPS**<sup>[*](https://docs.aws.amazon.com/ec2/latest/instancetypes/gp.html#gp_instance-store)</sup> without network bottlenecks
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

  • doesn't need to be link, can just be used as a reference section below

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

image

Comment thread docs/cloud/managed-postgres/overview.md Outdated

For detailed benchmark results, see [Performance benchmarks](/docs/cloud/managed-postgres/benchmarks).

For see [AWS local NVMe limits for details](https://docs.aws.amazon.com/ec2/latest/instancetypes/gp.html#gp_instance-store).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor

@achudnovskij achudnovskij left a comment

Choose a reason for hiding this comment

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

Added comments

@mintlify
Copy link
Copy Markdown

mintlify Bot commented May 26, 2026

Docs PR opened: ClickHouse/mintlify-docs-dev#68

Added an NVMe footnote, a Performance benchmarks link, and an AWS local NVMe limits link to the Managed Postgres overview.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 320e339. Configure here.

Comment thread docs/cloud/managed-postgres/overview.md Outdated
Comment thread docs/cloud/managed-postgres/overview.md Outdated
@yashpreetbathla yashpreetbathla merged commit 5812603 into main May 27, 2026
18 checks passed
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.

5 participants