[
https://issues.apache.org/jira/browse/LUCENE-4255?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13422803#comment-13422803
]
Chris Male commented on LUCENE-4255:
------------------------------------
Yup it's good! +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]
