Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add topic partitions to style guide #98

Merged
merged 5 commits into from
Dec 9, 2024
Merged

add topic partitions to style guide #98

merged 5 commits into from
Dec 9, 2024

Conversation

Deflaimun
Copy link
Contributor

No description provided.

@Deflaimun Deflaimun requested a review from a team as a code owner December 9, 2024 22:03
Copy link

netlify bot commented Dec 9, 2024

Deploy Preview for redpanda-documentation ready!

Name Link
🔨 Latest commit 35e0553
🔍 Latest deploy log https://app.netlify.com/sites/redpanda-documentation/deploys/67576f11137c4500087ca699
😎 Deploy Preview https://deploy-preview-98--redpanda-documentation.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 86 (🟢 up 23 from production)
Accessibility: 78 (🔴 down 1 from production)
Best Practices: 100 (🟢 up 25 from production)
SEO: 67 (🔴 down 8 from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@@ -1100,6 +1100,9 @@ Incorrect: To enable Shadow Indexing on a cluster, set the following parameters
|topic
|A topic is a stream of related events. A doc topic is content in Redpanda product doc library.
|
|topic partition
|Write without hyphens. A topic may be partitioned through multiple brokers. A "topic partition" represents this logical separation on Redpanda. It is managed by Raft, applying voting mechanisms such as Round Robin. In contrast, other Kafka applications use ZooKeeper to manage their partitions. In Redpanda this is managed natively by Raft.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
|Write without hyphens. A topic may be partitioned through multiple brokers. A "topic partition" represents this logical separation on Redpanda. It is managed by Raft, applying voting mechanisms such as Round Robin. In contrast, other Kafka applications use ZooKeeper to manage their partitions. In Redpanda this is managed natively by Raft.
|Write as 2 words, not 1 hyphenated word.

Copy link
Contributor

Choose a reason for hiding this comment

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

@paulo please also fix the formatting of the word list table while you're in this file!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

@micheleRP micheleRP left a comment

Choose a reason for hiding this comment

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

nice fix, thanks Paulo!

@Deflaimun Deflaimun merged commit d8035ec into main Dec 9, 2024
5 checks passed
@Deflaimun Deflaimun deleted the style-guide branch December 9, 2024 22:32
@Deflaimun
Copy link
Contributor Author

Wrote the definition here in the glossary entry
https://github.com/redpanda-data/docs/pull/909/files

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.

2 participants