activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From clebertsuconic <...@git.apache.org>
Subject [GitHub] activemq-artemis pull request #1787: ARTEMIS-1577 Address-settings policies ...
Date Thu, 18 Jan 2018 14:20:50 GMT
Github user clebertsuconic commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/1787#discussion_r162353815
  
    --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/postoffice/impl/PostOfficeImpl.java
---
    @@ -736,11 +736,11 @@ public RoutingStatus route(final Message message,
              throw new IllegalStateException("Message cannot be routed more than once");
           }
     
    -      setPagingStore(message);
    +      setPagingStore(context.getAddress(message), message);
     
           AtomicBoolean startedTX = new AtomicBoolean(false);
     
    -      final SimpleString address = message.getAddressSimpleString();
    +      final SimpleString address = context.getAddress(message);
    --- End diff --
    
    Your comment could apply to the previous version as well.. I didn't make any big changes
here. nor did want to risk anything else.


---

Mime
View raw message