hachikuji commented on code in PR #11969: URL: https://github.com/apache/kafka/pull/11969#discussion_r870644114
########## core/src/test/scala/unit/kafka/server/KafkaConfigTest.scala: ########## @@ -1509,4 +1512,29 @@ class KafkaConfigTest { assertNotNull(config.getLong(KafkaConfig.SaslOAuthBearerJwksEndpointRetryBackoffMsProp)) assertNotNull(config.getLong(KafkaConfig.SaslOAuthBearerJwksEndpointRetryBackoffMaxMsProp)) } + + @Test Review Comment: Hmm. I think the consequence is that the authorizer implementation has to be ready to receive authorization requests before the startup future has returned. At a minimum, I think we should update the documentation for `Authorizer.start` to mention the role of `early.start.listeners`. -- 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