divijvaidya commented on PR #13437:
URL: https://github.com/apache/kafka/pull/13437#issuecomment-1488637379

   Very interesting change! We can follow this pattern in many other places in 
the code base.
   
   However, I would like to understand first why we didn't consider usage of 
native Java's data structures such as CopyOnWrite* data structures. Also, 
please note that the usage of Immutable data structures is nice for cases with 
low write and high read throughput use cases. It would be nice if you could 
mention why the trade-off with immutable data structures is worth it in the 
case of AclAuthorizer.


-- 
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

Reply via email to