jbertram commented on PR #4584: URL: https://github.com/apache/activemq-artemis/pull/4584#issuecomment-1692229955
> So I guess its just a case of deciding whether we want to retain the guava-style inline execution behaviour, or change to the Caffeine async execution default. Regarding the cache usage in `org.apache.activemq.artemis.core.security.impl.SecurityStoreImpl` my thoughts are: 1. The only use-case where it really matters is with size = 0 and that's been fixed that already via #4589. 2. It makes sense to optimize security since it is used basically every time a client connects. Therefore, I'm in favor of using the default from Caffeine. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
