philipnee commented on code in PR #15228:
URL: https://github.com/apache/kafka/pull/15228#discussion_r1457944304
##########
clients/src/main/java/org/apache/kafka/common/telemetry/internals/ClientTelemetryReporter.java:
##########
@@ -362,7 +362,7 @@ public long timeToNextUpdate(long requestTimeoutMs) {
throw new IllegalStateException("Unknown telemetry state:
" + localState);
}
- log.debug("For telemetry state {}, returning the value {} ms; {}",
localState, timeMs, msg);
+ log.trace("For telemetry state {}, returning the value {} ms; {}",
localState, timeMs, msg);
Review Comment:
Do you think it makes sense to rephrase the log? For the `msg` can we
explicitly say:
...client will wait for {}ms before submitting the next...
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
