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

KAFKA-18589: Cleanup GroupMetadataManager #18619

Merged
merged 1 commit into from
Jan 19, 2025

Conversation

frankvicky
Copy link
Collaborator

@frankvicky frankvicky commented Jan 19, 2025

JIRA: KAFKA-18589
After #18468, interBrokerProtocolVersion is no longer used in GroupMetadataManager, we should remove it.

Committer Checklist (excluded from commit message)

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

After apache#18468, interBrokerProtocolVersion is no longer used in
GroupMetadataManager, we should remove it.
@github-actions github-actions bot added triage PRs from the community core Kafka Broker small Small PRs labels Jan 19, 2025
@frankvicky
Copy link
Collaborator Author

Hi @ijuma
Could you please take a look?
Many thanks 🙇🏼

@frankvicky frankvicky changed the title MINOR: Remove unused interBrokerProtocolVersion in GroupMetadataManager KAFKA-18589: Cleanup GroupMetadataManager Jan 19, 2025
@chia7712 chia7712 merged commit 485d36d into apache:trunk Jan 19, 2025
10 checks passed
chia7712 pushed a commit that referenced this pull request Jan 19, 2025
@github-actions github-actions bot removed the triage PRs from the community label Jan 20, 2025
pranavt84 pushed a commit to pranavt84/kafka that referenced this pull request Jan 27, 2025
airlock-confluentinc bot pushed a commit to confluentinc/kafka that referenced this pull request Jan 27, 2025
manoj-mathivanan pushed a commit to manoj-mathivanan/kafka that referenced this pull request Feb 19, 2025
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.

2 participants