ableegoldman commented on code in PR #19546:
URL: https://github.com/apache/kafka/pull/19546#discussion_r2059377283
##########
clients/src/main/java/org/apache/kafka/clients/consumer/KafkaConsumer.java:
##########
@@ -1776,10 +1777,13 @@ public void close() {
}
/**
- * Tries to close the consumer cleanly within the specified timeout. This
method waits up to
- * {@code timeout} for the consumer to complete pending commits and leave
the group.
+ * This method has been deprecated since Kafka 4.0 and should use {@link
KafkaConsumer#close(CloseOptions)} instead.
+ * <p>
+ * Close the consumer with {@link
CloseOptions.GroupMembershipOperation#DEFAULT default membership operation}
+ * cleanly within the specified timeout. This method waits up to
+ * {@code timeout} for the consumer to complete pending commits and may
leave the group (if applicable).
Review Comment:
agreed, "if applicable" is pretty vague and I'd be left wondering if I saw
this as a user
--
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
