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