Rancho-7 commented on code in PR #19728:
URL: https://github.com/apache/kafka/pull/19728#discussion_r2093810937


##########
clients/src/main/java/org/apache/kafka/clients/admin/ListShareGroupOffsetsResult.java:
##########
@@ -66,7 +66,8 @@ public KafkaFuture<Map<String, Map<TopicPartition, 
OffsetAndMetadata>>> all() {
     }
 
     /**
-     * Return a future which yields a map of topic partitions to offsets for 
the specified group.
+     * Return a future which yields a map of topic partitions to offsets for 
the specified group. If the group doesn't
+     * have a committed offset for a specific partition, the corresponding 
value in the returned map will be null.
      *
      * @param groupId The group ID.

Review Comment:
   Yeah,I think so,but totally okay if you prefer keeping this.



-- 
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.

To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org

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

Reply via email to