[GitHub] [activemq-artemis] clebertsuconic commented on a change in pull request #2790: ARTEMIS-2449 limit size of producer details

[GitBox <gi...@apache.org>]

clebertsuconic 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_r312154013
 
 

 ##########
 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:
   Will this represent any error in case the size > 100 or this is just for information?
   
   If I send to a destination that was cleaned before, will I be able to send it again?
   
   
   I believe this is just for informational purposes on the Console?

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services