michaelandrepearce commented on a change in pull request #2790: ARTEMIS-2449 
limit size of producer details
URL: https://github.com/apache/activemq-artemis/pull/2790#discussion_r312129457
 
 

 ##########
 File path: 
artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/ServerSessionImpl.java
 ##########
 @@ -175,7 +176,7 @@
    private final OperationContext context;
 
    // Session's usage should be by definition single threaded, hence it's not 
needed to use a concurrentHashMap here
-   protected final Map<SimpleString, Pair<Object, AtomicLong>> 
targetAddressInfos = new HashMap<>();
+   protected final Map<SimpleString, Pair<Object, AtomicLong>> 
targetAddressInfos = new MaxSizeMap<>(100);
 
 Review comment:
   Can this be configurable, 100 is a bit arbitrary especially as prev was 
unlimited

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to