Mailing-List: contact dev-help@activemq.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@activemq.apache.org
From: gaohoward <git@git.apache.org>
To: dev@activemq.apache.org
Reply-To: dev@activemq.apache.org
References: <git-pr-705-activemq-artemis@git.apache.org>
In-Reply-To: <git-pr-705-activemq-artemis@git.apache.org>
Subject: [GitHub] activemq-artemis issue #705: ARTEMIS-670 Adding destination creation and del...
Content-Type: text/plain
Message-Id: <20160809145255.E6D5DDFE1A@git1-us-west.apache.org>
Date: Tue,  9 Aug 2016 14:52:55 +0000 (UTC)
archived-at: Tue, 09 Aug 2016 14:52:58 -0000

Github user gaohoward commented on the issue:

    https://github.com/apache/activemq-artemis/pull/705
  
    @clebertsuconic that kinda goes back to my original impl, which combine all the functionalities into one command, i.e. 
    
    ./artemis destination create|delete --name  myqueue1 --core
    
    which is not so good in that some options are irrelevant for certain operations, causing confusion. for example delete a queue is just need a name, the filter, jndi bindings, etc are not relevant. If people do
    
    ./artemis help destination 
    
    it will print all the options together. Having separated commands can reduce the confusion, because each command has only relevant options.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---