[ 
https://issues.apache.org/jira/browse/MAHOUT-408?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12874928#action_12874928
 ] 

Drew Farris commented on MAHOUT-408:
------------------------------------

Thanks for the patch Shannon.

The patch applies cleanly, build completes and all unit tests pass. -h and 
--help options now work for all mentioned operations. 

The approach to handing transpose seems suitable.

This looks good to commit. Any other comments?  



> Mahout CLI OptionExceptions when given "-h" or "--help" for specific programs
> -----------------------------------------------------------------------------
>
>                 Key: MAHOUT-408
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-408
>             Project: Mahout
>          Issue Type: Bug
>    Affects Versions: 0.4
>            Reporter: Shannon Quinn
>            Priority: Minor
>             Fix For: 0.4
>
>         Attachments: MAHOUT-408.patch
>
>
> The following Mahout programs incorrectly processed -h flags, throwing 
> OptionExceptions:
> seq2sparse
> seqdirectory
> seqwiki
> svd
> trainclassifier
> transpose
> The first five are easily addressed by adding "parser.setHelpOption(helpOpt)" 
> in the correct driver files (thanks to Jeff Eastman). The "transpose" program 
> also contained a NullPointerException thrown when two integer arguments were 
> not verified. See the patch file for a possible fix.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to