activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From michaelandrepearce <...@git.apache.org>
Subject [GitHub] activemq-artemis pull request #1736: ARTEMIS-1569 - Queue - User Enhancement
Date Thu, 21 Dec 2017 14:50:14 GMT
Github user michaelandrepearce commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/1736#discussion_r158294873
  
    --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/ActiveMQServerImpl.java
---
    @@ -2508,7 +2508,7 @@ private void deployQueuesFromListCoreQueueConfiguration(List<CoreQueueConfigurat
                 // if the address::queue doesn't exist then create it
                 try {
                    createQueue(SimpleString.toSimpleString(config.getAddress()), config.getRoutingType(),
    -                           queueName, SimpleString.toSimpleString(config.getFilterString()),null,
    +                           queueName, SimpleString.toSimpleString(config.getFilterString()),
SimpleString.toSimpleString(config.getUser()),
    --- End diff --
    
    @clebertsuconic it does, it was existing functionality internally, just not set-able from
broker.xml I have added a test just to prove that it is persisted and survives restart


---

Mime
View raw message