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

    https://github.com/apache/activemq-artemis/pull/1775#discussion_r162269219
  
    --- Diff: 
artemis-core-client/src/main/java/org/apache/activemq/artemis/core/server/QueueQueryResult.java
 ---
    @@ -55,13 +57,19 @@ public QueueQueryResult(final SimpleString name,
                                final int consumerCount,
                                final long messageCount,
                                final boolean autoCreateQueues,
    +                           final boolean autoCreateQueuesDurable,
                                final boolean exists,
                                final boolean autoCreated,
                                final boolean purgeOnNoConsumers,
                                final RoutingType routingType,
                                final int maxConsumers) {
    +
           this.durable = durable;
     
    +      if (autoCreateQueues) {
    +         this.durable = autoCreateQueuesDurable;
    --- End diff --
    
    As noted this should be an enum, and only override, if enum is not null, 
currently his just overrides the value with no option to not override.


---

Reply via email to