[
https://issues.apache.org/jira/browse/ARTEMIS-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17762169#comment-17762169
]
ASF subversion and git services commented on ARTEMIS-4417:
----------------------------------------------------------
Commit aa4e4409bf2cf4b14a02b5b2887b900e30cae84d in activemq-artemis's branch
refs/heads/main from Robbie Gemmell
[ https://gitbox.apache.org/repos/asf?p=activemq-artemis.git;h=aa4e4409bf ]
ARTEMIS-4417: use ConcurrentHashMap for the sub-maps to make their use thread
safe
> AbstractJournalStorageManager storeKeyValuePair + deleteKeyValuePair are not
> thread safe
> ----------------------------------------------------------------------------------------
>
> Key: ARTEMIS-4417
> URL: https://issues.apache.org/jira/browse/ARTEMIS-4417
> Project: ActiveMQ Artemis
> Issue Type: Bug
> Components: Balancer
> Affects Versions: 2.30.0
> Reporter: Robbie Gemmell
> Priority: Major
> Fix For: 2.31.0
>
>
> Continuing on from ARTEMIS-4406, the calls to AbstractJournalStorageManager
> storeKeyValuePair + deleteKeyValuePair used by LocalCache as part of the
> connection-router implementation are themselves also not entirely thread safe.
> A ConcurrentHashMap is used for the initial structure, however a regular
> HashMap sub-map is then stored within the concurrent outer map, which can
> then also be accessed concurrently, meaning the methods aren't currently
> thread safe; it also needs to be a ConcurrentHashMap.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
