phoenix-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hadoop QA (JIRA)" <>
Subject [jira] [Commented] (PHOENIX-3517) Use argparse in
Date Sun, 18 Dec 2016 06:40:59 GMT


Hadoop QA commented on PHOENIX-3517:

{color:red}-1 overall{color}.  Here are the results of testing the latest attachment
  against master branch at commit 006beddec042c1642261a79d632b987491e7d361.
  ATTACHMENT ID: 12843737

    {color:green}+1 @author{color}.  The patch does not contain any @author tags.

    {color:green}+0 tests included{color}.  The patch appears to be a documentation, build,
                        or dev patch that doesn't require tests.

    {color:green}+1 javac{color}.  The applied patch does not increase the total number of
javac compiler warnings.

    {color:red}-1 javadoc{color}.  The javadoc tool appears to have generated 44 warning messages.

    {color:green}+1 release audit{color}.  The applied patch does not increase the total number
of release audit warnings.

    {color:red}-1 lineLengths{color}.  The patch introduces the following lines longer than
    +parser.add_argument('url', nargs='?', help='The URL to the Phoenix Query Server.', default='http://localhost:8765')
+parser.add_argument('-u', '--user', help='Username for database authentication (unsupported).',
+parser.add_argument('-p', '--password', help='Password for database authentication (unsupported).',
+parser.add_argument('-a', '--authentication', help='Mechanism for HTTP authentication.',
choices=('SPNEGO', 'BASIC', 'DIGEST', 'NONE'), default='')
+parser.add_argument('-s', '--serialization', help='Serialization type for HTTP API.', choices=('PROTOBUF',
'JSON'), default=None)

    {color:green}+1 core tests{color}.  The patch passed unit tests in .

Test results:
Javadoc warnings:
Console output:

This message is automatically generated.

> Use argparse in
> -------------------------------
>                 Key: PHOENIX-3517
>                 URL:
>             Project: Phoenix
>          Issue Type: Improvement
>            Reporter: Josh Elser
>            Assignee: Josh Elser
>             Fix For: 4.10.0
>         Attachments: PHOENIX-3517.002.patch, PHOENIX-3517.patch
> just does hand-rolled argument parsing. We can simplify the code, make
better usage and validation choices, and support some extra options that we presently default
> Python's argparse does all of these for us.

This message was sent by Atlassian JIRA

View raw message