skaundinya15 commented on a change in pull request #10743:
URL: https://github.com/apache/kafka/pull/10743#discussion_r657960058



##########
File path: 
clients/src/main/java/org/apache/kafka/clients/admin/internals/CoordinatorKey.java
##########
@@ -24,7 +24,7 @@
     public final String idValue;

Review comment:
       Yeah I can definitely propose that in the discussion thread there. As I 
was working on my implementation on KIP-709 for batching the 
`listConsumerGroupsOffsets` API, I saw the need to make this class public since 
it will be needed to handle the `ListConsumerGroupsOffsetsResult` class 
properly. But I'll take this discussion to the email thread/the PR I'll open 
shortly.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to