Skip to content

Conversation

@harshini-rangaswamy
Copy link
Contributor

@harshini-rangaswamy harshini-rangaswamy commented Nov 25, 2025

Describe your changes

Clarifies single-AZ BYOC Kafka risks, usage guidance, and SLA requirements. Improves section formatting and terminology.

FLEET-6228

Checklist

  • The first paragraph of the page is on one line.
  • The other lines have a line break at 90 characters.
  • I checked the output.
  • I applied the style guide.
  • My links start with /docs/.

@cloudflare-workers-and-pages
Copy link

Deploying aiven-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: b5badb1
Status: ✅  Deploy successful!
Preview URL: https://65f196cd.aiven-docs.pages.dev
Branch Preview URL: https://fleet-6228-clarify-data-loss.aiven-docs.pages.dev

View logs

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR improves the Apache Kafka best practices documentation by clarifying guidance on single-AZ deployments for BYOC customers. The changes enhance readability through clearer, more concise language and expand the single-AZ section with detailed risk assessments, replication factor implications, and usage recommendations.

  • Enhanced single-AZ section with comprehensive guidance on risks, appropriate use cases, and SLA requirements
  • Improved clarity and conciseness across all sections by simplifying technical explanations
  • Updated title to better reflect the document's purpose

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +121 to +122
To enable this option for your project, contact [Aiven support](mailto:[email protected])
or your account team.
Copy link

Copilot AI Nov 25, 2025

Choose a reason for hiding this comment

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

Line length exceeds 90 characters: According to the PR checklist, lines should have a line break at 90 characters. This line exceeds that limit. Consider breaking it before "contact":

To enable this option for your project, contact
[Aiven support](mailto:[email protected]) or your account team.
Suggested change
To enable this option for your project, contact [Aiven support](mailto:[email protected])
or your account team.
To enable this option for your project,
contact [Aiven support](mailto:[email protected]) or your account team.

Copilot uses AI. Check for mistakes.
A maximum of 4,000 partitions per broker and 200,000 per cluster is recommended. For more
details, see this [Apache Kafka blog post](https://blogsarchive.apache.org/kafka/entry/apache-kafka-supports-more-partitions).
In addition, the total number of topics per cluster should remain under 7,000.
A maximum of 4,000 partitions per broker and 200,000 per cluster is recommended. For details, see this
Copy link

Copilot AI Nov 25, 2025

Choose a reason for hiding this comment

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

Line length exceeds 90 characters: According to the PR checklist, lines should have a line break at 90 characters. This line appears to exceed that limit. Consider breaking it after "recommended" or "details":

A maximum of 4,000 partitions per broker and 200,000 per cluster is recommended. For
details, see this [Apache Kafka blog post]...
Suggested change
A maximum of 4,000 partitions per broker and 200,000 per cluster is recommended. For details, see this
A maximum of 4,000 partitions per broker and 200,000 per cluster is recommended.
For details, see this

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants