Skip to content

MINOR: Fix typo and docs#20373

Merged
FrankYang0529 merged 2 commits intoapache:trunkfrom
joshua2519:minor-fix-typos
Aug 19, 2025
Merged

MINOR: Fix typo and docs#20373
FrankYang0529 merged 2 commits intoapache:trunkfrom
joshua2519:minor-fix-typos

Conversation

@joshua2519
Copy link
Contributor

@joshua2519 joshua2519 commented Aug 19, 2025 •
edited by github-actions bot
Loading

Fix typo and docs in following.

clients/src/main/java/org/apache/kafka/clients/consumer/ConsumerRebalanceListener.java
clients/src/main/resources/common/message/FetchRequest.json
raft/src/main/java/org/apache/kafka/raft/KafkaRaftClient.java

Reviewers: Kuan-Po Tseng brandboat@gmail.com, Lan Ding
isDing_L@163.com, Ken Huang s7133700@gmail.com, TengYao Chi
kitingiao@gmail.com, Chia-Ping Tsai chia7712@gmail.com, PoAn Yang
payang@apache.org

@github-actions github-actions bot added triage PRs from the community consumer kraft clients small Small PRs labels Aug 19, 2025
FrankYang0529
FrankYang0529 approved these changes Aug 19, 2025
View reviewed changes
Copy link
Member

@FrankYang0529 FrankYang0529 left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for the patch.

brandboat
brandboat approved these changes Aug 19, 2025
View reviewed changes
Copy link
Member

@brandboat brandboat left a comment

Choose a reason for hiding this comment

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

LGTM!

DL1231
DL1231 approved these changes Aug 19, 2025
View reviewed changes
Copy link
Collaborator

@DL1231 DL1231 left a comment

Choose a reason for hiding this comment

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

Thansk for the patch, LGTM!

frankvicky
frankvicky reviewed Aug 19, 2025
View reviewed changes
Copy link
Contributor

@frankvicky frankvicky left a comment

Choose a reason for hiding this comment

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

Please fix the build error.

m1a2st
m1a2st approved these changes Aug 19, 2025
View reviewed changes
Copy link
Collaborator

@m1a2st m1a2st left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM

@FrankYang0529 FrankYang0529 merged commit 38c3a41 into apache:trunk Aug 19, 2025
25 checks passed
@github-actions github-actions bot removed the triage PRs from the community label Aug 20, 2025
@joshua2519 joshua2519 deleted the minor-fix-typos branch November 26, 2025 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@frankvicky frankvicky frankvicky left review comments

@FrankYang0529 FrankYang0529 FrankYang0529 approved these changes

@chia7712 chia7712 chia7712 approved these changes

+3 more reviewers

@brandboat brandboat brandboat approved these changes

@DL1231 DL1231 DL1231 approved these changes

@m1a2st m1a2st m1a2st approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

ci-approved clients consumer kraft small Small PRs

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@joshua2519 @brandboat @FrankYang0529 @chia7712 @frankvicky @DL1231 @m1a2st