[ https://issues.apache.org/jira/browse/HDFS-12002?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16056630#comment-16056630 ]
Chen Liang edited comment on HDFS-12002 at 6/20/17 11:15 PM: ------------------------------------------------------------- Post initial patch. Thanks [~elek] for the comments! Added the change you suggested. was (Author: vagarychen): Post initial patch. Thanks [~elek] for the comments! Added the change you suggested. Also, I want to take back the #4 change I mentioned in the description, as ozone is still under development, it seems more beneficial to print the exception trace message rather than hiding them for the time being. > Ozone : SCM cli misc fixes/improvements > --------------------------------------- > > Key: HDFS-12002 > URL: https://issues.apache.org/jira/browse/HDFS-12002 > Project: Hadoop HDFS > Issue Type: Sub-task > Reporter: Chen Liang > Assignee: Chen Liang > Fix For: ozone > > Attachments: HDFS-12002-HDFS-7240.001.patch > > > Currently there are a few minor issues with the SCM CLI: > 1. some commands do not use -c option to take container name. an issue with > this is that arguments need to be in a certain order to be correctly parsed, > e.g.: > {{./bin/hdfs scm -container -del c0 -f}} works, but > {{./bin/hdfs scm -container -del -f c0}} will not > A more important thing is that, since -del requires the following argument > being container name, if someone types {{./bin/hdfs scm -container -del > -help}} it will be an error, while we probably want to display a help message > instead. > 2.some subcommands are not displaying the errors in the best way it could be, > e.g.: > {{./bin/hdfs scm -container -del}} is wrong because it misses container name. > So cli complains > {code} > Missing argument for option: del > Unrecognized options:[-container, -del] > usage: hdfs scm <commands> [<options>] > where <commands> can be one of the following > -container Container related options > {code} > but this does not really show that it is container name it is missing > 3. probably better to rename -del to -delete to be consistent with other > commands like -create and -info -- This message was sent by Atlassian JIRA (v6.4.14#64029) --------------------------------------------------------------------- To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org