C0urante opened a new pull request, #13351: URL: https://github.com/apache/kafka/pull/13351
[Jira](https://issues.apache.org/jira/browse/KAFKA-14781) Instead of logging an `ERROR`-level message when topic syncing is enabled (which it is by default) and no ACL authorizer is configured on the source Kafka cluster, we first issue an `INFO`-level message that suggests to the user that they disable topic ACL syncing, and then for each subsequent time, issue a `DEBUG`-level message. Some thoughts while implementing this: 1. The number of testing-only constructors for `MirrorSourceConnector` is getting a bit out of hand. We might consider replacing all of them with a package-private `start(MirrorSourceConfig)` method, which can help unify testing and non-testing code paths, as well as reduce the testing-only code that we have to tack on to non-testing code. 2. We could possibly expand on the test case introduced here to simulate an ACL authorizer being added to the source Kafka cluster during the lifetime of the connector. I've omitted this as I don't believe it's a very common case and the changes here do not seem likely to introduce any risk for it. ### Committer Checklist (excluded from commit message) - [ ] Verify design and implementation - [ ] Verify test coverage and CI build status - [ ] Verify documentation (including upgrade notes) -- 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
