Skip to content

Rephrase replication factor documentation #3909

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

Merged
merged 2 commits into from
May 19, 2025

Conversation

sh-cho
Copy link
Contributor

@sh-cho sh-cho commented May 17, 2025

  • (1) Fix docs in RetryableTopic - The sentence was not properly separated. I think it was confusing.
  • (2) Repharased docs in RetryTopicConfigurationBuilder to match the wording used in other sections, e.g. change history
    The default replication factor for the retry topics is now `-1` (use broker default).
    If your broker is earlier that version 2.4, you will now need to explicitly set the property.

sh-cho added 2 commits May 18, 2025 01:27
Signed-off-by: Seonghyeon Cho <seonghyeoncho96@gmail.com>
Signed-off-by: Seonghyeon Cho <seonghyeoncho96@gmail.com>
@sobychacko sobychacko added this to the 4.0.0-M3 milestone May 19, 2025
@sobychacko sobychacko merged commit ec154a3 into spring-projects:main May 19, 2025
3 checks passed
spring-builds pushed a commit that referenced this pull request May 19, 2025
* Fix docs on `RetryableTopic.replicationFactor()`

* Rephrase replicationFactor description about broker version

Signed-off-by: Seonghyeon Cho <seonghyeoncho96@gmail.com>

(cherry picked from commit ec154a3)
spring-builds pushed a commit that referenced this pull request May 19, 2025
* Fix docs on `RetryableTopic.replicationFactor()`

* Rephrase replicationFactor description about broker version

Signed-off-by: Seonghyeon Cho <seonghyeoncho96@gmail.com>

(cherry picked from commit ec154a3)
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