Github user franz1981 commented on the pull request:
https://github.com/apache/activemq-artemis/commit/dfa70680fed37d25aa3a6d0d6a0795e580495b6a#commitcomment-31769915
No need to synchronize it: `stores` is a concurrent hashmap and you can use
the atomic `computeIfAbsent` that use a
GitHub user gaohoward opened a pull request:
https://github.com/apache/activemq-artemis/pull/2478
ARTEMIS-2210 PagingStore creation is not properly synchronized
In PagingManagerImpl#getPageStore() the operations on the map 'stores'
are not synchronzed and it's possible that more
Github user michaelandrepearce commented on the issue:
https://github.com/apache/activemq-artemis/pull/2444
@jbertram my concerns have been addressed
---