[
https://issues.apache.org/jira/browse/AVRO-836?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13046178#comment-13046178
]
Philip Zeyliger commented on AVRO-836:
--------------------------------------
This looks great to me. Some comments:
1) Are you sure you want to put avro into scripts/? Seems like src/avro would
be just as good a place.
2) Usage help:
{quote}
[1]air::py(129882)$PYTHONPATH=src ./scripts/avro
error: You must specify `cat` or `write`
[1]air::py(129888)$PYTHONPATH=src ./scripts/avro write
error: No schema specified
{quote}
Have you considered spitting out a more useful message here? You can get
optparse to spit out fairly nice error messages. It's hard to figure out
what's required and how to use it.
Thanks!
> Have an "avro" command line utility to display and write Avro files
> -------------------------------------------------------------------
>
> Key: AVRO-836
> URL: https://issues.apache.org/jira/browse/AVRO-836
> Project: Avro
> Issue Type: New Feature
> Components: python
> Reporter: Miki Tebeka
> Labels: command-line, python
> Attachments: AVRO-836.diff
>
>
> Following email conversation, this is a port of
> [avrocat|https://bitbucket.org/tebeka/avrocat/src] to the avro code base.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
