Uploaded image for project: 'Lucene - Core'
  1. Lucene - Core
  2. LUCENE-4255

clean up analysis factory exception handling

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 4.0-BETA, 6.0
    • None
    • None
    • New

    Description

      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.

      Attachments

        1. LUCENE-4255.patch
          74 kB
          Robert Muir

        Activity

          People

            Unassigned Unassigned
            rcmuir Robert Muir
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: