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, 04 Dec 2016 00:09:58 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 683e1f0fe8f61e6c01f766a9d25ca3e953691464.
  ATTACHMENT ID: 12841649

    {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.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