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