You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@phoenix.apache.org by "Geoffrey Jacoby (Jira)" <ji...@apache.org> on 2022/06/15 21:28:00 UTC

[jira] [Resolved] (PHOENIX-6554) Pherf CLI option long/short option names do not follow conventions

     [ https://issues.apache.org/jira/browse/PHOENIX-6554?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Geoffrey Jacoby resolved PHOENIX-6554.
--------------------------------------
    Fix Version/s: 5.2.0
     Release Note: Pherf CLI now allows its long option names to use -- in addition to -, as the Unix convention suggests. While the existing behavior of using a single dash "-" for long options doesn't follow the Unix convention, we still allow it for backward compatibility. 
       Resolution: Fixed

Merged to master. Thanks [~ankurmahe]!

> Pherf CLI option long/short option names do not follow conventions
> ------------------------------------------------------------------
>
>                 Key: PHOENIX-6554
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-6554
>             Project: Phoenix
>          Issue Type: Improvement
>          Components: core
>    Affects Versions: 5.2.0
>            Reporter: Istvan Toth
>            Assignee: Ankur Maheshwari
>            Priority: Minor
>             Fix For: 5.2.0
>
>
> The Pherf script does not use long and short option names consistently.
> for example:
> -t and --thin are for specifying the the thin PQS URL, 
> and 
> -z and --zookeeper are for the ZK quorum
> but 
> -schemaFile is used to specify the schema file, and 
> --schemaFile does not work.
> IMO options that look like long options should also be accepted with double dash, or we could just invent new short options  for them (which would break backwards compatibility).
> i.e. instead of 
> {code:java}
> options.addOption("schemaFile", true,
>                 "Regex or file name for the Test phoenix table schema .sql to use.");
> {code}
> we could have one of the following:
> {code:java}
> options.addOption("sf", "schemaFile", true,
>                 "Regex or file name for the Test phoenix table schema .sql to use.");
> options.addOption("schemaFile", "schemaFile", true,
>                 "Regex or file name for the Test phoenix table schema .sql to use.");
> {code}



--
This message was sent by Atlassian Jira
(v8.20.7#820007)