Adrian Cole created ARTEMIS-2054:
------------------------------------

             Summary: Artemis appears to never issue a JMS callback on error
                 Key: ARTEMIS-2054
                 URL: https://issues.apache.org/jira/browse/ARTEMIS-2054
             Project: ActiveMQ Artemis
          Issue Type: Improvement
          Components: Broker
    Affects Versions: 2.6.2
         Environment: I'm trying to use artemis for Brave (zipkin java tracer) 
testing of JMS 2.0 implementation.

I've noticed that the completion listener is never called on error. This might 
be a concern beyond my testing. Here's how I found this..

I traced the use of CompletionListener in 
org.apache.activemq.artemis.jms.client.ActiveMQMessageProducer. At what's 
currently line 515, it is used, but wrapped with a type that will never call 
the callback.

{{         if (completionListener != null) {}}
{{            clientProducer.send(address, coreMessage, new 
CompletionListenerWrapper(completionListener, jmsMessage, this));}}
{{         } else {}}
{{            clientProducer.send(address, coreMessage);}}
{{         }}}

{{Also, all catch blocks won't call the callback either. It is possible I 
missed something.}}
            Reporter: Adrian Cole


I'm trying to use artemis for Brave (zipkin java tracer) testing of JMS 2.0 
implementation.

I've noticed that the completion listener is never called on error. This might 
be a concern beyond my testing. Here's how I found this..

 

I traced the use of CompletionListener in 
org.apache.activemq.artemis.jms.client.ActiveMQMessageProducer. At what's 
currently line 515, it is used, but wrapped with a type that will never call 
the callback.

{{code}}
         if (completionListener != null) {
            clientProducer.send(address, coreMessage, new 
CompletionListenerWrapper(completionListener, jmsMessage, this));
         } else {
            clientProducer.send(address, coreMessage);
         }
{{code}}

Also, all catch blocks won't call the callback either.

 

 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to