Skip to content

KAFKA-16866: Used the right constant in RLMT#testFetchQuotaManagerConfig #16152

Merged
chia7712 merged 1 commit intoapache:trunkfrom
kamalcph:KAFKA-16866
May 31, 2024
Merged

KAFKA-16866: Used the right constant in RLMT#testFetchQuotaManagerConfig #16152
chia7712 merged 1 commit intoapache:trunkfrom
kamalcph:KAFKA-16866

Conversation

@kamalcph
Copy link
Copy Markdown
Contributor

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

@kamalcph kamalcph requested a review from chia7712 May 31, 2024 10:32
@kamalcph
Copy link
Copy Markdown
Contributor Author

This is a minor follow-up of #16146

@kamalcph kamalcph requested a review from showuon May 31, 2024 10:33
Copy link
Copy Markdown
Member

@chia7712 chia7712 left a comment

Choose a reason for hiding this comment

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

@kamalcph thanks for this fix. Sorry that I had a rough PR :_

@chia7712 chia7712 merged commit cdd4455 into apache:trunk May 31, 2024
wernerdv pushed a commit to wernerdv/kafka that referenced this pull request Jun 3, 2024
…QuotaManagerConfig (apache#16152)

Reviewers: Chia-Ping Tsai <chia7712@gmail.com>
@kamalcph kamalcph deleted the KAFKA-16866 branch June 4, 2024 06:57
TaiJuWu pushed a commit to TaiJuWu/kafka that referenced this pull request Jun 8, 2024
…QuotaManagerConfig (apache#16152)

Reviewers: Chia-Ping Tsai <chia7712@gmail.com>
satishd pushed a commit that referenced this pull request Jun 11, 2024
…QuotaManagerConfig (#16152)

Reviewers: Chia-Ping Tsai <chia7712@gmail.com>
gongxuanzhang pushed a commit to gongxuanzhang/kafka that referenced this pull request Jun 12, 2024
…QuotaManagerConfig (apache#16152)

Reviewers: Chia-Ping Tsai <chia7712@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants