Skip to content

KAFKA-19558: Batch consumer group descriptions - #23060

Open
lh0156 wants to merge 1 commit into
apache:trunkfrom
lh0156:agent/KAFKA-19558-batch-describe-groups
Open

KAFKA-19558: Batch consumer group descriptions#23060
lh0156 wants to merge 1 commit into
apache:trunkfrom
lh0156:agent/KAFKA-19558-batch-describe-groups

Conversation

@lh0156

@lh0156 lh0156 commented Aug 3, 2026

Copy link
Copy Markdown

Summary

  • Split consumer group describe requests into batches of at most 50 groups.
  • Preserve the existing result aggregation and output behavior for individual and all-group descriptions.
  • Keep the change scoped to describe operations, which avoids changing the existing offset reset behavior.

Tests

  • RED: ConsumerGroupServiceTest.testDescribeConsumerGroupsIsBatched failed before the change because 101 group IDs were sent in one request.
  • GREEN: ConsumerGroupServiceTest.testDescribeConsumerGroupsIsBatched
  • ConsumerGroupServiceTest
  • DescribeConsumerGroupTest.testDescribeAllExistingGroups
  • Checkstyle, SpotBugs, and Spotless

Split consumer group describe requests into bounded batches so --describe --all-groups remains usable when a cluster has many groups.

Generated-by: OpenAI Codex (GPT-5)

Signed-off-by: Yunseop Eom <62834176+lh0156@users.noreply.github.com>
@github-actions github-actions Bot added triage PRs from the community tools small Small PRs labels Aug 3, 2026
@lh0156

lh0156 commented Aug 3, 2026

Copy link
Copy Markdown
Author

Revalidated the current head locally without changing the patch:

  • ./gradlew :tools:test --tests org.apache.kafka.tools.consumer.group.ConsumerGroupServiceTest --tests org.apache.kafka.tools.consumer.group.DescribeConsumerGroupTest :tools:checkstyleMain :tools:checkstyleTest :tools:spotbugsMain :tools:spotlessCheck --no-build-cache --console=plain
  • Result: BUILD SUCCESSFUL (112 actionable tasks)
  • The batching regression test and the existing DescribeConsumerGroupTest integration coverage passed.

No code changes were made in this verification pass.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

small Small PRs tools triage PRs from the community

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant