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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to