search

strimzi / strimzi-kafka-operator

Apache Kafka® running on Kubernetes
https://strimzi.io/
Apache License 2.0
4.77k stars 1.28k forks source link

Use grouping by resource when managing Acl rules #724

Closed scholzj closed 5 years ago

scholzj commented 6 years ago

The SimpleAclAuthorizer accepts the rules which are created / removed in following format Map<Resource, Set<Acl>>. It allows to group the ACLs per resource and call the KAfka methods only few times.

In the User Operator we currently call this separately for every single Acl rule (i.e. our Set<Acl> has always size 1). I wonder if there might be some speedup in organizing the Acl rules per resource and making few calls to Zookeeper. I expect that it migth decrease the traffic anf put less load mainly on Zookeeper.

scholzj commented 5 years ago

This should be fixed in #813 and should be closed.