Github user michaelandrepearce commented on the pull request:
https://github.com/apache/activemq-artemis/commit/a2da41ee2e347bcf8fe721bded89e4a78ca14cfb#commitcomment-29899679
In
artemis-jms-client/src/main/java/org/apache/activemq/artemis/jms/client/ActiveMQMessageProducer.java:
In
artemis-jms-client/src/main/java/org/apache/activemq/artemis/jms/client/ActiveMQMessageProducer.java
on line 574:
The new response handler only gets invoked for correlating ids, Also the
dedupe is the reason for the cache, e.g. it should only via the cache which
"should" take care of this, by the nature that the id is removed from cache, so
next will have cache miss and do nothing.
Was there a case it was being invoked twice? If so that should be fixed as
would be a bug in the cache code , or somewhere its being called not via the
cache
---
