gemmellr commented on PR #4584: URL: https://github.com/apache/activemq-artemis/pull/4584#issuecomment-1691855301
> See #4589. With this PR rebased after those changes went in, I retested these changes with and without the final commit making the eviction processing behave more like Guava's. As expected, both ways now pass the previously-failing test, now that the likely underlying issue of 'temporarily caching when max size is 0' issue was resolved by the other PR omitting the cache in that situation. 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. -- 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]
