Details

    • Sub-task
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 4.0-ALPHA
    • modules/analysis
    • None
    • New

    Description

      With Analyzer now using TokenStreamComponents, theres no reason for Analysis consumers to use tokenStream() (it just gives bad performance). Consequently all consumers will be moved over to using reusableTokenStream(). The only challenge here is that reusableTokenStream throws an IOException which many consumers are not rigged to deal with.

      Once all consumers have been moved, we can rename reusableTokenStream() back to tokenStream().

      Attachments

        1. LUCENE-3455.patch
          12 kB
          Chris Male
        2. LUCENE-3455-test-consumers.patch
          43 kB
          Chris Male
        3. LUCENE-3455-test-consumers.patch
          37 kB
          Chris Male

        Activity

          People

            cmale Chris Male
            cmale Chris Male
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment