C0urante commented on code in PR #13475: URL: https://github.com/apache/kafka/pull/13475#discussion_r1200807046
########## config/connect-mirror-maker.properties: ########## @@ -57,3 +57,6 @@ config.storage.replication.factor=1 # replication.policy.separator = _ # sync.topic.acls.enabled = false # emit.heartbeats.interval.seconds = 5 + +# enable flow in the logs for improved diagnostics +add.flow.context = true Review Comment: I left my thoughts on the discussion thread, but TL;DR: I think disabling by default both programmatically and in relevant config files is the safest (and follows precedent set by prior KIPs to added connector contexts to log files), and we can add a commented-out line to relevant config files that would enable the new log context behavior and add an explanation above it with our recommendations to users to un-comment it if they're starting new clusters or if their logging infrastructure is prepared to handle the changes it would cause. -- 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
