cmccabe opened a new pull request #10505: URL: https://github.com/apache/kafka/pull/10505
Fix some cases where ControllerApis was blocking on the controller thread. This should not be necessary, since the controller thread can just interface directly with the network threads. alterClientQuotas and incrementalAlterConfigs were not doing authorization correctly in ControllerApis.scala. Since the previous release of KRaft did not support authorizers, this bug is not as severe as it could have been, but it still needs to be fixed. This PR also adds unit tests to verify that all of the controller operations return authorization failures when appropriate. Additionally, this PR fixes a comment in ControllerApis#deleteTopics that no longer reflects what the code is doing when we don't have "describe" permission. -- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org