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
---
