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

Billie Rinaldi commented on ACCUMULO-846:
-----------------------------------------

It's a better practice to throw exceptions so information about what went wrong 
is propagated, so the methods should do that when possible.  Iterators should 
also make sure to check the boolean returned by super.validateOptions.
                
> Options validation is inconsistent
> ----------------------------------
>
>                 Key: ACCUMULO-846
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-846
>             Project: Accumulo
>          Issue Type: Bug
>          Components: client
>            Reporter: Billie Rinaldi
>            Assignee: Billie Rinaldi
>             Fix For: 1.5.0
>
>
> The validateOptions method is used to validate iterator options when 
> configuring them via the setiter command in the shell.  It is implemented 
> inconsistently, sometimes throwing an exception to indicate bad options and 
> sometimes catching the exception and returning false.

--
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