franz1981 commented on a change in pull request #2520: ARTEMIS-2238 Enhancement
to queueQuery on producer
URL: https://github.com/apache/activemq-artemis/pull/2520#discussion_r251038639
##########
File path:
artemis-jms-client/src/main/java/org/apache/activemq/artemis/jms/client/ActiveMQSession.java
##########
@@ -397,6 +380,55 @@ public MessageProducer createProducer(final Destination
destination) throws JMSE
}
}
+ void checkDestination(ActiveMQDestination destination) throws JMSException {
+ SimpleString address = destination.getSimpleAddress();
+ // TODO: What to do with FQQN
+ if (!connection.containsKnownDestination(address)) {
+ try {
Review comment:
i will pack this slow path in a separate method to make the code more
readable (perf are helped as well, but is not relevant)!
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
