[ 
https://issues.apache.org/jira/browse/HBASE-16620?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15487664#comment-15487664
 ] 

Ted Yu commented on HBASE-16620:
--------------------------------

{code}
87                + " -t              table name. If specified, only backup 
images, which contain this table\n "
{code}
The above should start with '-t table' to match synopsis.
{code}
+        if (tables == null) {
+          System.out.println("ERROR: Backup set '" + setName+ "' is either 
empty or does not exist");
{code}
System.err should be used when exit code is negative, right ?
{code}
+      } catch(IllegalArgumentException e) {
+        System.out.println("ERROR: Illegal argument for backup root path: "+ 
value);
{code}
Add space between catch and (.
Please use System.err



> Command-line tool usability issues
> ----------------------------------
>
>                 Key: HBASE-16620
>                 URL: https://issues.apache.org/jira/browse/HBASE-16620
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Vladimir Rodionov
>            Assignee: Vladimir Rodionov
>         Attachments: HBASE-16620-v1.patch
>
>
> We need to address issues found by [~saint....@gmail.com]
> https://issues.apache.org/jira/browse/HBASE-7912?focusedCommentId=15484865&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15484865



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to