clebertsuconic commented on PR #4540: URL: https://github.com/apache/activemq-artemis/pull/4540#issuecomment-1624271282
I'm running the whole test suite with these changes (on a separate CI). If caffeine has a bigger community, the jar is smaller, I think we should bring this right away (if tests are ok of course). my initial concern was about the community support and CVEs.. but from what I later gathered this is actually better. performance wise I think the differences would be impossible to measure. We currently use some login changes from Security as Justin mentioned... but you would have to use the cache quite a lot to have any significant result. I think we should merge this, but not much for the performance reasons. anyone has anything against that? -- 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]
