LoganZhuZzz commented on code in PR #19320:
URL: https://github.com/apache/kafka/pull/19320#discussion_r2022164085
##########
clients/src/main/java/org/apache/kafka/common/ClusterResourceListener.java:
##########
@@ -24,7 +24,7 @@
* <p>
* <h4>Clients</h4>
* There will be one invocation of {@link
ClusterResourceListener#onUpdate(ClusterResource)} after each metadata response.
- * Note that the cluster id may be null when the Kafka broker version is below
0.10.1.0. If you receive a null cluster id, you can expect it to always be null
unless you have a cluster with multiple broker versions which can happen if the
cluster is being upgraded while the client is running.
+ * Note the minimum supported broker version is 2.1. If you receive a null
cluster id, you can expect it to always be null unless you have a cluster with
multiple broker versions which can happen if the cluster is being upgraded
while the client is running.
Review Comment:
> It is impossible to have "null" cluster id for now, right?
I will remove the outdated part.
--
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
