Rancho-7 commented on code in PR #19728:
URL: https://github.com/apache/kafka/pull/19728#discussion_r2099014700
##########
clients/src/main/java/org/apache/kafka/clients/admin/ListConsumerGroupOffsetsResult.java:
##########
@@ -60,9 +58,6 @@ public KafkaFuture<Map<TopicPartition, OffsetAndMetadata>>
partitionsToOffsetAnd
* Return a future which yields a map of topic partitions to
OffsetAndMetadata objects 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:
I think we should keep the `@param` part. WDYT?
##########
clients/src/main/java/org/apache/kafka/clients/admin/ListShareGroupOffsetsResult.java:
##########
@@ -68,9 +68,6 @@ public KafkaFuture<Map<String, Map<TopicPartition,
OffsetAndMetadata>>> all() {
/**
* 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:
ditto
##########
clients/src/main/java/org/apache/kafka/clients/admin/ListStreamsGroupOffsetsResult.java:
##########
@@ -52,9 +52,6 @@ public KafkaFuture<Map<String, Map<TopicPartition,
OffsetAndMetadata>>> all() {
/**
* 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:
ditto
--
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
