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

Keith Turner commented on ACCUMULO-1358:
----------------------------------------

Thanks for the patch Mike.  I applied it to 1.5 and trunk..  Will need to leave 
the ticket open until we determine if this will be in 1.5.0 or 1.5.1.  I had to 
make the unit test delete the table it created, otherwise a later unit test 
would fail.  The unit test probably ran in a different order for you.

bq. Added unit tests for illegal arguments in setiter. Did not add tests for 
valid classes because the command then expects user input and I wasn't sure how 
to wire that.

org.apache.accumulo.test.ShellServerTest test setting a valid class, so this 
case is covered.  
                
> Shell's setiter is not informative when using a bad class name
> --------------------------------------------------------------
>
>                 Key: ACCUMULO-1358
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-1358
>             Project: Accumulo
>          Issue Type: Bug
>          Components: shell
>            Reporter: John Vines
>            Assignee: Mike Drob
>              Labels: newbie
>             Fix For: 1.5.0
>
>         Attachments: ACCUMULO-1358.v1.patch
>
>
> In the shell, I did setiter using a class that wasn't found. Rather then a 
> message about it not being found, I just get told that I have an invalid 
> argument. Even turning on debug, I had to use the stack trace to figure out 
> why it was erroring.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to