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

    https://github.com/apache/activemq-artemis/pull/924#discussion_r92668152
  
    --- Diff: 
artemis-cli/src/main/java/org/apache/activemq/artemis/cli/commands/queue/UpdateQueue.java
 ---
    @@ -31,7 +31,7 @@
        String name;
     
        @Option(name = "--deleteOnNoConsumers", description = "whether to 
delete when it's last consumers disconnects)")
    -   Boolean deleteOnNoConsumers = null;
    +   String deleteOnNoConsumers = null;
    --- End diff --
    
    @clebertsuconic You only get -1 like every one else ;) 
    
    TBH I care less about the mechanism and more about the abiliity to update a 
queue.  If there's precedence for this case already set, then fair enough.  
@franz1981 probably best to be consistent for the time being.  I would like to 
see more effort put into the CLI in general, IMO it needs some care and 
attention, but we can address that at a later date.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to