Skip to content
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

MINOR: Remove unused member in DynamicBrokerConfig #18915

Merged
merged 1 commit into from
Feb 16, 2025

Conversation

frankvicky
Copy link
Contributor

Remove some unused members that are left out by #18790

Committer Checklist (excluded from commit message)

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

@github-actions github-actions bot added triage PRs from the community core Kafka Broker small Small PRs labels Feb 15, 2025
Copy link
Contributor

@Yunyung Yunyung left a comment

Choose a reason for hiding this comment

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

LGTM :)

@chia7712
Copy link
Member

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

@chia7712 chia7712 merged commit 5cbe00e into apache:trunk Feb 16, 2025
9 of 10 checks passed
@github-actions github-actions bot removed the triage PRs from the community label Feb 17, 2025
manoj-mathivanan pushed a commit to manoj-mathivanan/kafka that referenced this pull request Feb 19, 2025
Reviewers: Jhen-Yung Hsu <jhenyunghsu@gmail.com>, 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
ci-approved core Kafka Broker small Small PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants