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 migration tutorials for broker load balancing #657

Merged
merged 4 commits into from
Aug 3, 2023

Conversation

Anonymitaet
Copy link
Member

@Anonymitaet Anonymitaet commented Jul 27, 2023

What change does this PR introduce?

This PR adds migration tutorials for broker load balancing (topics in the red box below)

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 @D-2-Ed many thanks for your technical guidance and great support!

@Anonymitaet Anonymitaet added this to the 3.1.0 milestone Jul 27, 2023
@Anonymitaet Anonymitaet self-assigned this Jul 27, 2023
@github-actions github-actions bot added the doc Improvements or additions to documentation label Jul 27, 2023
@@ -35,7 +35,8 @@
"concepts-broker-load-balancing-features",
"concepts-broker-load-balancing-benefits",
"concepts-broker-load-balancing-concepts",
"concepts-broker-load-balancing-types"
"concepts-broker-load-balancing-types",
"concepts-broker-load-balancing-migration"
Copy link
Member Author

Choose a reason for hiding this comment

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

For reviewers: I know here is not the perfect place, but there is no better place to locate this tutorial, so I put all types of docs under the same subject "Broker Load Balancing". I'll adjust it once there is a better solution.

image

@Anonymitaet Anonymitaet changed the title [DO NOT REVIEW][feat][doc] add migration tutorials for broker load balancing [feat][doc] add migration tutorials for broker load balancing Jul 31, 2023
@Anonymitaet
Copy link
Member Author

The preview looks good:

image

@Anonymitaet
Copy link
Member Author

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

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

Thank you all!

Copy link
Member

@Demogorgon314 Demogorgon314 left a comment

Choose a reason for hiding this comment

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

LGTM. @heesung-sn Can you help review this PR?

@Anonymitaet
Copy link
Member Author

@heesung-sn do you have more comments? If no, I'll merge this PR.

@Anonymitaet Anonymitaet merged commit 0dc0656 into apache:main Aug 3, 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.

3 participants