[GitHub] [kafka] emissionnebula commented on pull request #13282: KAKFA-14733: Added a few missing checks for Kraft Authorizer and updated AclAuthorizerTest to run tests for both zk and kraft

["emissionnebula (via GitHub)" <gi...@apache.org>]

emissionnebula commented on PR #13282:
URL: https://github.com/apache/kafka/pull/13282#issuecomment-1438402661

   Thanks @omkreddy for the review. I have updated the description. 
   
   Also, the build seems to be failing due to unrelated test failures. 
   - `org.apache.kafka.connect.mirror.integration.MirrorConnectorsIntegrationBaseTest.testReplication()` - java.lang.RuntimeException: Could not find enough records. found 0, expected 1
   - `org.apache.kafka.connect.mirror.integration.DedicatedMirrorIntegrationTest.testMultiNodeCluster()` - org.opentest4j.AssertionFailedError: Condition not met within timeout 30000. topic A.test-topic-0 was not created on cluster B- ._~:/?#[]@!$&'()*+;="<>%{}|\^‘618 in time ==> expected: <true> but was: <false>


-- 
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-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org