Skip to content

Conversation

@TaiJuWu
Copy link
Collaborator

@TaiJuWu TaiJuWu commented Dec 2, 2025

Base on #21039, Implement list
offset RPC support topicId for consumer part.

- Test building request with topic IDs for version 12
- Test building request with topic names for version 11
- Test version 12 requires topic IDs (throws exception without them)
- Test version 11 requires topic names (throws exception without them)
- Test canUseTopicIds parameter limits max version
- Update testGetErrorResponse to handle both version 11 and 12
…pport

Added tests for ListOffsets API version 12 topic ID support:
- testHandleListOffsetRequestWithUnknownTopicId: Tests that unknown topic IDs return UNKNOWN_TOPIC_ID error
- testHandleListOffsetRequestVersion11WithTopicName: Tests backward compatibility with version 11 using topic names
- testHandleListOffsetRequestWithTopicIdDisabled: Placeholder test for topic ID resolution (currently disabled due to metadata cache issue)

These tests verify the changes made in handleListOffsetRequest to support:
1. Topic ID resolution in version >= 12
2. Returning UNKNOWN_TOPIC_ID error for unresolvable topic IDs
3. Backward compatibility with version < 12 using topic names
@github-actions github-actions bot added triage PRs from the community core Kafka Broker consumer storage Pull requests that target the storage module clients labels Dec 2, 2025
@TaiJuWu TaiJuWu changed the title (WIP) KAFKA-10549: Add topic ID support to ListOffsets, OffsetForLeaders - consumer (2/N) (WIP) KAFKA-10549: Add topic ID support to ListOffsets - consumer (2/N) Dec 2, 2025
@TaiJuWu TaiJuWu force-pushed the KAFKA-10549-listOffset-consumer branch from 053ba6f to ef854ab Compare December 4, 2025 00:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clients consumer core Kafka Broker storage Pull requests that target the storage module triage PRs from the community

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant