rondagostino commented on pull request #10550:
URL: https://github.com/apache/kafka/pull/10550#issuecomment-827040926
> we need to start forwarding the ACL operations to the controller
We forward the Create/Remove operations to the controller, BUT this patch
actually short-circuits that if we are using KRaft with the ZooKeeper-based
`AclAuthorizer` via the changes to `RaftSupport.maybeForward()`. The reason
for short-circuiting it is because the KRaft controller doesn't have the code
to create or remove ACLs (`handle{Create,Delete}Acls` in `KafkaApis`). We
could add it, of course, in which case the changes to the `maybeForward()`
method would be unnecessary. Perhaps it would be simpler to do that instead of
delaying it to an additional PR -- is that what you were suggesting?
--
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
