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

[feat][doc] add use cases + features + benefits + types for broker load balancing #652

Merged
merged 2 commits into from
Jul 25, 2023

Conversation

Anonymitaet
Copy link
Member

What change does this PR introduce?

This PR adds the following conceptual docs for broker load balancing (topics in the red box below)

  • Use cases
  • Features
  • Benefits
  • Types
image

Doc development plan - Read before review

For the current doc of Broker Load Balancer, we want to tackle the following challenges:

  • Create new documentation to cover all aspects of the feature (especially for the new extensible load balancer).
  • Fill in any content gaps that may exist.
  • Consolidate existing documentation into the new IA, as it is currently scattered and disorganized.
  • Organize the documentation in a cohesive manner, ensuring it flows logically.

Consequently, we’ve created the following content development plans:

Ackowledgements

@Demogorgon314 @heesung-sn @D-2-Ed many thanks for your technical guidance and great support!

@github-actions github-actions bot added the doc Improvements or additions to documentation label Jul 23, 2023
@Anonymitaet
Copy link
Member Author

Previews go as expected:

image image image

image

@Anonymitaet
Copy link
Member Author

@Demogorgon314 @heesung-sn @D-2-Ed this doc has been reviewed by you in Broker Load Balancing | Features + Benefits + Types + Use Cases, can you take a quick look at this PR?

@momo-jun @Huanli-Meng could you please help review this PR?

Thank you all!

@Anonymitaet Anonymitaet added this to the 3.1.0 milestone Jul 23, 2023
Copy link
Contributor

@Huanli-Meng Huanli-Meng left a comment

Choose a reason for hiding this comment

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

only 2 minor comments.

docs/concepts-broker-load-balancing-benefits.md Outdated Show resolved Hide resolved
docs/concepts-broker-load-balancing-types.md Show resolved Hide resolved
@Demogorgon314 Demogorgon314 self-requested a review July 24, 2023 09:42
@Anonymitaet Anonymitaet merged commit c357c43 into apache:main Jul 25, 2023
2 checks passed
@Technoboy- Technoboy- mentioned this pull request Aug 9, 2023
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants