Add DescribeConsumer support in Topic SDK#760
Conversation
There was a problem hiding this comment.
Pull request overview
Adds support for the TopicService DescribeConsumer API in the Python Topic SDK so users can fetch consumer details (including optional per-partition stats and location info).
Changes:
- Added
describe_consumer()to both async and sync Topic clients. - Introduced new public request/response types for DescribeConsumer, plus corresponding grpcwrapper parsing/to_public conversions.
- Added control-plane tests covering DescribeConsumer basics and offset/stat assertions.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
ydb/topic.py |
Exposes describe_consumer() on async/sync clients and exports TopicConsumerDescription. |
ydb/_grpc/grpcwrapper/ydb_topic_public_types.py |
Adds DescribeConsumer request params and public result dataclasses for consumer/partition stats and location. |
ydb/_grpc/grpcwrapper/ydb_topic.py |
Implements protobuf-to-public conversions for DescribeConsumerResult, PartitionConsumerStats, and PartitionLocation. |
ydb/_apis.py |
Registers DescribeConsumer method name constant for TopicService calls. |
tests/topics/test_control_plane.py |
Adds async/sync tests for DescribeConsumer, including stats and committed offsets. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
bcfeeb0 to
5e8c7b2
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
5e8c7b2 to
bedcf7b
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Implements DescribeConsumer API call for Topic SDK, allowing users to get detailed information about a topic's consumer including partition stats and consumer offsets. Closes #713
bedcf7b to
4e23719
Compare
Implements DescribeConsumer API call for Topic SDK, allowing users to get detailed information about a topic's consumer including partition stats and consumer offsets.
Closes #713
Pull request type
Please check the type of change your PR introduces:
What is the current behavior?
Issue Number: N/A
What is the new behavior?
Other information