AndrewJSchofield commented on code in PR #17431:
URL: https://github.com/apache/kafka/pull/17431#discussion_r1793500761
##########
clients/src/main/java/org/apache/kafka/common/telemetry/internals/ClientTelemetryReporter.java:
##########
@@ -623,14 +622,7 @@ public void initiateClose(long timeoutMs) {
return;
}
- try {
- log.info("About to wait {} ms. for terminal telemetry push
to be submitted", timeoutMs);
Review Comment:
You don't actually make use of the `timeoutMs` argument any more (apart from
in a debug log line). Probably ought to remove it.
--
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
