[GitHub] [kafka] cmccabe commented on pull request #10550: MINOR: Add support for ZK Authorizer with KRaft

2021-05-11 Thread GitBox
cmccabe commented on pull request #10550: URL: https://github.com/apache/kafka/pull/10550#issuecomment-839281487 Thanks for working on this, @rondagostino , and sorry about the delays in reviewing. > We forward the Create/Remove operations to the controller, but this patch actually

[GitHub] [kafka] cmccabe commented on pull request #10550: MINOR: Add support for ZK Authorizer with KRaft

2021-04-19 Thread GitBox
cmccabe commented on pull request #10550: URL: https://github.com/apache/kafka/pull/10550#issuecomment-822694203 Thanks for tackling this. Please keep in mind that this is a plugin architecture. The ZK-based authorizer shouldn't get any special treatment, nor should its setup be