clolov commented on code in PR #18432:
URL: https://github.com/apache/kafka/pull/18432#discussion_r1913029564
##########
core/src/test/scala/unit/kafka/server/KafkaApisTest.scala:
##########
@@ -485,44 +412,6 @@ class KafkaApisTest extends Logging {
}
}
- @Test
- def testAlterConfigsWithAuthorizer(): Unit = {
Review Comment:
Or to phrase it differently, I am unclear as to why we used to do
authorisation in two places (once on the broker and once at the controller) and
now we are doing only one authorisation (only on the controller). We used to do
double authorisation even in Zookeeper-world, no? In other words, where we are
forwarding the request to is (somewhat) orthogonal to how many times we
authorise
--
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]
