Skip to content

MINOR: Fix typo in ApiKeyVersionsProvider exception message #19521

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

Conversation

apalan60
Copy link
Contributor

@apalan60 apalan60 commented Apr 20, 2025

This patch addresses issue #19516 and corrects a typo in
ApiKeyVersionsProvider: when toVersion exceeds latestVersion, the
IllegalArgumentException message was erroneously formatted with
fromVersion. The format argument has been updated to use toVersion
so that the error message reports the correct value.

Reviewers: Ken Huang s7133700@gmail.com, PoAn Yang
payang@apache.org, Jhen-Yung Hsu jhenyunghsu@gmail.com, Chia-Ping
Tsai chia7712@gmail.com

@github-actions github-actions bot added triage PRs from the community tests Test fixes (including flaky tests) clients small Small PRs labels Apr 20, 2025
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 for this quick fix, LGTM

@github-actions github-actions bot removed the triage PRs from the community label Apr 21, 2025
@chia7712
Copy link
Member

the hanging test is traced by https://issues.apache.org/jira/browse/KAFKA-19180

@chia7712 chia7712 merged commit 8fa0d97 into apache:trunk Apr 21, 2025
25 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-approved clients small Small PRs tests Test fixes (including flaky tests)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants