Skip to content

Fix typo 'amd' -> 'and' in CompletableEventReaperTest comments#22358

Closed
RonGamzu wants to merge 1 commit into
apache:trunkfrom
RonGamzu:fix/typo-amd-in-completable-event-reaper-test
Closed

Fix typo 'amd' -> 'and' in CompletableEventReaperTest comments#22358
RonGamzu wants to merge 1 commit into
apache:trunkfrom
RonGamzu:fix/typo-amd-in-completable-event-reaper-test

Conversation

@RonGamzu
Copy link
Copy Markdown
Contributor

Three test method comments in CompletableEventReaperTest contained a
copy-paste typo where "amd" was written instead of "and". This fixes all
three occurrences in testExpired, testCompleted, and
testCompletedAndExpired.

No logic changes — comment-only fix.

Three identical test comments in CompletableEventReaperTest contained
the typo 'amd' where 'and' was intended. Fixed all three occurrences
in testExpired, testCompleted, and testCompletedAndExpired.
@github-actions github-actions Bot added triage PRs from the community consumer tests Test fixes (including flaky tests) clients small Small PRs labels May 23, 2026
Copy link
Copy Markdown
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.

If these PRs only contains typo fixes, we can merge the two patches together.
#22358
#22359

And also update the title to use the MINOR: prefix.

@RonGamzu
Copy link
Copy Markdown
Contributor Author

RonGamzu commented May 30, 2026

@m1a2st Superseded by #22427, which consolidates this fix together with the related typo fix in LeaderEpochFileCache.

@RonGamzu RonGamzu closed this May 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clients consumer small Small PRs tests Test fixes (including flaky tests)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants