[GitHub] [activemq-artemis] clebertsuconic commented on issue #2800: ARTEMIS-2451 eliminate knownDestinations cache

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

clebertsuconic commented on issue #2800: ARTEMIS-2451 eliminate knownDestinations cache
URL: https://github.com/apache/activemq-artemis/pull/2800#issuecomment-522376248
 
 
   @jbertram Can I ask you to add a test with this PR?
   
   One that shows users doing this anti pattern:
   
   ```java
        int N = 100; // or any arbitrary number > 1
        for (int i = 0; i < N; i++) {
             Queue queue = session.createQueue("myQueue");
             MessageProducer prod = session.createProducer(queue);
             prod.send(....);
             prod.close();
        }
   ```
   
   If an user did such anti-pattern, it should suffer the consequences of the extra roundtrip of course, and we should not optimize for it.
   
   
   I just want to make sure it wouldn't fail for any reason.

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