hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-16982) Better integrate Apache CLI in AbstractHBaseTool
Date Tue, 08 Nov 2016 23:16:00 GMT

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

Hudson commented on HBASE-16982:
--------------------------------

FAILURE: Integrated in Jenkins build HBase-Trunk_matrix #1940 (See [https://builds.apache.org/job/HBase-Trunk_matrix/1940/])
HBASE-16982 Better integrate Apache CLI in AbstractHBaseTool. - (appy: rev 52241c90e4eab19b6613278d2e8107ab56c8a57a)
* (edit) hbase-server/src/test/java/org/apache/hadoop/hbase/snapshot/TestExportSnapshot.java
* (edit) hbase-common/src/main/java/org/apache/hadoop/hbase/util/AbstractHBaseTool.java
* (edit) hbase-server/src/main/java/org/apache/hadoop/hbase/snapshot/ExportSnapshot.java
* (add) hbase-common/src/test/java/org/apache/hadoop/hbase/util/AbstractHBaseToolTest.java


> Better integrate Apache CLI in AbstractHBaseTool
> ------------------------------------------------
>
>                 Key: HBASE-16982
>                 URL: https://issues.apache.org/jira/browse/HBASE-16982
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Appy
>            Assignee: Appy
>             Fix For: 2.0.0
>
>         Attachments: HBASE-16982.master.001.patch, HBASE-16982.master.002.patch, HBASE-16982.master.003.patch,
HBASE-16982.master.004.patch, HBASE-16982.master.005.patch
>
>
> Problem
> 1. Inconsistencies in our user facing tools.
> - options:
>     start with single/double dash
>     words separated by dash or underscore or just joined together
>     someplace use '=' to separate value, other use space (ExportSnapshot vs HashTable)
> - Description
>   Manually formatting options and their descriptions in printUsage()
>   Inconsistant formatting, sometimes even weird.
>   Incomplete. Sometimes people forget to add new option to description
> 2. Manual parsing of options (those huge if-else loops iterating over args)
> Solution
> Use Apache CLI
> - It has various validations for option names which'll fix first set of issues.
> - using AbstractHBaseTool's print usage function will ensure consistent formatting (although
we loose the power to order the options)
> - If we enforce the method of defining options as in patch, it's highly unlikely to forget
adding description.
> - CLI parses the options for us.
> Using Apache CLI when writing new tools is straight forward, but it's not easy when porting
exiting tools since some option names are not valid as per CLI's validation.
> New method, processOldArgs(), will allow to port these tools in a backward compatible
manner.



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

Mime
View raw message