C0urante commented on code in PR #11782:
URL: https://github.com/apache/kafka/pull/11782#discussion_r902584655
##########
connect/runtime/src/main/java/org/apache/kafka/connect/runtime/AbstractWorkerSourceTask.java:
##########
@@ -397,7 +399,11 @@ boolean sendRecords() {
producerRecord,
(recordMetadata, e) -> {
if (e != null) {
- log.debug("{} failed to send record to {}: ",
AbstractWorkerSourceTask.this, topic, e);
+ if (producerClosed) {
Review Comment:
Added this check to clean up the flood of `ERROR`-level log messages that
occurs when the task's producer is closed while there are still in-flight
messages.
This issue was not specific to these integration tests or to KIP-618, but it
clogged up the logs for these tests badly enough that a small tweak in the code
base to address it seemed warranted.
--
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]
