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

Dmitriy V. Ryaboy commented on PARQUET-13:
------------------------------------------

The PR is in https://github.com/apache/incubator-parquet-mr/pull/11

[~julienledem] do I need special git access? My attempts to follow the docs 
resulted in this:

{code}
Merge complete (local ref PR_TOOL_MERGE_PR_11_MASTER). Push to apache? (y/n): y
Username for 'https://git-wip-us.apache.org': dvryaboy
Password for 'https://[email protected]': 
fatal: Authentication failed
Restoring head pointer to 9ad5485c
Note: checking out '9ad5485c'.
{code}

(btw the script assumes a remote called apache-github while dev/README.md says 
github-apache...)

> The `-d` option for `parquet-schema` shouldn't have optional argument
> ---------------------------------------------------------------------
>
>                 Key: PARQUET-13
>                 URL: https://issues.apache.org/jira/browse/PARQUET-13
>             Project: Parquet
>          Issue Type: Bug
>          Components: parquet-mr
>            Reporter: Cheng Lian
>            Priority: Trivial
>
> According to 
> [{{ShowSchemaCommand.java}}|https://github.com/Parquet/parquet-mr/blob/parquet-1.5.0/parquet-tools/src/main/java/parquet/tools/command/ShowSchemaCommand.java#L78],
>  the {{-d | --detailed}} option doesn't require an optional argument. The 
> [{{hasOptionalArg()}}|https://github.com/Parquet/parquet-mr/blob/parquet-1.5.0/parquet-tools/src/main/java/parquet/tools/command/ShowSchemaCommand.java#L47]
>  call should be removed.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to