borisssmidtCET opened a new pull request, #14597: URL: https://github.com/apache/kafka/pull/14597
*More detailed description of your change, This PR makes the ACL Authorizer significantly faster. // before ``` Benchmark (aclCount) (authorizerType) (denyPercentage) (resourceCount) Mode Cnt Score Error Units AuthorizerBenchmark.testAclsIterator 500 ACL 50 5000 avgt 7 278.920 ± 28.651 ms/op AuthorizerBenchmark.testAuthorizeByResourceType 500 ACL 50 5000 avgt 7 4.614 ± 1.076 ms/op AuthorizerBenchmark.testAuthorizer 500 ACL 50 5000 avgt 7 0.185 ± 0.045 ms/op AuthorizerBenchmark.testUpdateCache 500 ACL 50 5000 avgt 7 641.337 ± 123.786 ms/op ``` // after ``` Benchmark (aclCount) (authorizerType) (denyPercentage) (resourceCount) Mode Cnt Score Error Units AuthorizerBenchmark.testAclsIterator 500 ACL 50 5000 avgt 7 243.929 ± 32.787 ms/op AuthorizerBenchmark.testAuthorizeByResourceType 500 ACL 50 5000 avgt 7 3.833 ± 0.375 ms/op AuthorizerBenchmark.testAuthorizer 500 ACL 50 5000 avgt 7 0.033 ± 0.002 ms/op AuthorizerBenchmark.testUpdateCache 500 ACL 50 5000 avgt 7 595.364 ± 25.375 ms/op JMH benchmarks done ``` *Summary of testing strategy (including rationale) Since the code is equivalent the no extra tests should be added. ### Committer Checklist (excluded from commit message) - [ ] Verify design and implementation - [ ] Verify test coverage and CI build status - [ ] Verify documentation (including upgrade notes) -- 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-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
