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

Chris Male commented on LUCENE-4255:
------------------------------------

I originally added InitializationException as a way to have consistent 
Exceptions thrown since some Factories threw IAEs, others through 
SolrExceptions, others just generic RuntimeExceptions.  If we're going to 
consistently use IAEs, I'm all for it +1
                
> clean up analysis factory exception handling
> --------------------------------------------
>
>                 Key: LUCENE-4255
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4255
>             Project: Lucene - Core
>          Issue Type: Bug
>            Reporter: Robert Muir
>         Attachments: LUCENE-4255.patch
>
>
> I don't like wrapping everything in the InitializationException, or that 
> inform() gets a resourceloader (whose methods throw IOExceptions), but every 
> analysis factory that implements this then needs to wrap the IOExceptions as 
> something else.
> I think its way more intuitive to use IllegalArgumentException in init(args), 
> and change inform() to throw IOException.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to