[GitHub] [incubator-kyuubi] pan3793 commented on pull request #826: [KYUUBI #825] Introduce jcommander framework to kyuubi project

[GitBox <gi...@apache.org>]

pan3793 commented on pull request #826:
URL: https://github.com/apache/incubator-kyuubi/pull/826#issuecomment-882550421


   After second thought and have a discussion with @turboFei (author of this the first version of `kyuubi-ctl`),  I change my vote to `kyuubi-ctl <create|get|delete|list> <server|engine> options`. It's because
   1. some tools use `cmd action object`, i.e. `kubectl`, some tools chose `cmd subcmd subcmd`, both are widely used.
   2. one of the fact is that `kyuubi-ctl list service` and `kyuubi-ctl list engine` are almost the same things except for the `znode root path`, and we can reuse lots code if chose the pattern `cmd action object`


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@kyuubi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org