Sebastian T created ARTEMIS-2571:
------------------------------------

             Summary: Remove unneccessary synchronization in ActiveMQServerImpl
                 Key: ARTEMIS-2571
                 URL: https://issues.apache.org/jira/browse/ARTEMIS-2571
             Project: ActiveMQ Artemis
          Issue Type: Improvement
          Components: Broker, Web Console
    Affects Versions: 2.10.1
            Reporter: Sebastian T


The ActiveMQServerImpl sessions field is a ConcurrentHashMap. Synchronizing on 
the ActiveMQServerImpl object to iterate over the map is not necessary. 
ActiveMQServerImpl#getSession, ActiveMQServerImpl#removeSession and 
ActiveMQServerImpl#createSession also work on the sessions field without 
synchronizing on the ActiveMQServerImpl.

Removing the synchronized keyword on the ActiveMQServerImpl#getSessions() 
methods e.g. improves loading of Connections view, especially when multiple 
administrators are using the UI.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to