Github user clebertsuconic commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/2254#discussion_r211419574
  
    --- Diff: 
artemis-jms-client/src/main/java/org/apache/activemq/artemis/jms/client/ActiveMQMessage.java
 ---
    @@ -371,7 +369,7 @@ public Destination getJMSReplyTo() throws JMSException {
                 String name = address.toString();
     
                 // swap the old prefixes for the new ones so the proper 
destination type gets created
    -            if (enable1xPrefixes) {
    +            if (session.isEnable1xPrefixes()) {
    --- End diff --
    
    It's the same.. instead of a boolean per message, I'm using a single 
property on the session now. 
    
    We can improve it later to make sure it's static, but I can't do it ATM 
becuase of RA and caches of connections.


---

Reply via email to