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

Jay Kreps commented on KAFKA-1291:
----------------------------------

Hey [~sgeller] this is great! I updated this patch a bit:
1. I removed the shell script for a couple of tools that seemed kind of 
esoteric and seemed more likely to add confusion.
2. I made all commands print a one-line explanation of what they do when run 
without arguments. Let me know if you think this description seems reasonable.
3. I refactored a little bit of the cut-and-paste in our command line args 
definition.

This should be an easy patch to review as most changes are trivial.

> Make wrapper shell scripts for important tools
> ----------------------------------------------
>
>                 Key: KAFKA-1291
>                 URL: https://issues.apache.org/jira/browse/KAFKA-1291
>             Project: Kafka
>          Issue Type: Improvement
>    Affects Versions: 0.8.1
>            Reporter: Jay Kreps
>              Labels: newbie, usability
>             Fix For: 0.8.2
>
>         Attachments: KAFKA-1291.patch, KAFKA-1291.patch, KAFKA-1291.patch
>
>
> It is nice to have a proper command for the important tools just to help with 
> discoverability. I noticed that mirror maker doesn't have such a wrapper. 
> Neither does consumer offset checker. It would be good to do an audit and 
> think of any tools that should have a wrapper that don't.



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

Reply via email to