xiaodongdu commented on a change in pull request #8691:
URL: https://github.com/apache/kafka/pull/8691#discussion_r428205238
##########
File path:
connect/runtime/src/main/java/org/apache/kafka/connect/runtime/Worker.java
##########
@@ -92,6 +93,7 @@
private final ExecutorService executor;
private final Time time;
private final String workerId;
+ private final String clusterId;
Review comment:
I'm still keeping clusterId as a class variable in Worker, since we are
using clusterId in other method of this class. We are getting clusterId from
ConnectUtils.lookupKafkaClusterId inside Worker constructor and keep it as a
class variable value.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
