AndrewJSchofield commented on code in PR #19477:
URL: https://github.com/apache/kafka/pull/19477#discussion_r2047604943
##########
clients/src/main/java/org/apache/kafka/clients/admin/Admin.java:
##########
@@ -879,20 +879,24 @@ default DescribeConsumerGroupsResult
describeConsumerGroups(Collection<String> g
/**
* List the consumer groups available in the cluster.
+ * @deprecated Since 4.1. Use {@link Admin#listGroups(ListGroupsOptions)}
instead.
*
* @param options The options to use when listing the consumer groups.
* @return The ListConsumerGroupsResult.
*/
+ @Deprecated
ListConsumerGroupsResult listConsumerGroups(ListConsumerGroupsOptions
options);
/**
* List the consumer groups available in the cluster with the default
options.
* <p>
* This is a convenience method for {@link
#listConsumerGroups(ListConsumerGroupsOptions)} with default options.
* See the overload for more details.
+ * @deprecated Since 4.1. Use {@link Admin#listGroups(ListGroupsOptions)}
instead.
Review Comment:
I don't think so. If you want `Admin.listGroups()` to return just consumer
groups, you need to apply a filter to group type and protocol type.
--
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
