AntonRoskvist commented on PR #4023: URL: https://github.com/apache/activemq-artemis/pull/4023#issuecomment-1104767472
@clebertsuconic thanks for the feedback Okay... at this point I'm basically just moving the current "session-level" destination cache and making it shared, but the way this cache is implemented now makes collisions happen so often that I'm not really sure it's worth it to be honest. I'm mostly doing this for a use case I have with a very large number of queues and chances of a cache-clash is really high already at "just" some few hundreds of destinations. If you prefer I change this (and I can do that for sure), do you think it makes sense to just store all destinations in something like a hashmap instead that just holds all destinations, or should I revert this particular part of the change to it's old behavior, i.e Session-level? Either way is fine for me. -- 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]
