Lucene - Core
  1. Lucene - Core
  2. LUCENE-3396

Make TokenStream Reuse Mandatory for Analyzers


    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.0-ALPHA
    • Component/s: modules/analysis
    • Labels:
    • Lucene Fields:


      In LUCENE-2309 it became clear that we'd benefit a lot from Analyzer having to return reusable TokenStreams. This is a big chunk of work, but its time to bite the bullet.

      I plan to attack this in the following way:

      • Collapse the logic of ReusableAnalyzerBase into Analyzer
      • Add a ReuseStrategy abstraction to Analyzer which controls whether the TokenStreamComponents are reused globally (as they are today) or per-field.
      • Convert all Analyzers over to using TokenStreamComponents. I've already seen that some of the TokenStreams created in tests need some work to be reusable (even if they aren't reused).
      • Remove Analyzer.reusableTokenStream and convert everything over to using .tokenStream (which will now be returning reusable TokenStreams).
      1. LUCENE-3396-remaining-merging.patch
        181 kB
        Chris Male
      2. LUCENE-3396-remaining-analyzers.patch
        43 kB
        Chris Male
      3. LUCENE-3396-rab.patch
        101 kB
        Chris Male
      4. LUCENE-3396-rab.patch
        103 kB
        Chris Male
      5. LUCENE-3396-rab.patch
        104 kB
        Chris Male
      6. LUCENE-3396-rab.patch
        104 kB
        Chris Male
      7. LUCENE-3396-rab.patch
        99 kB
        Chris Male
      8. LUCENE-3396-rab.patch
        102 kB
        Chris Male
      9. LUCENE-3396-rab.patch
        100 kB
        Chris Male
      10. LUCENE-3396-forgotten.patch
        9 kB
        Chris Male

        Issue Links


          No work has yet been logged on this issue.


            • Assignee:
              Chris Male
              Chris Male
            • Votes:
              0 Vote for this issue
              1 Start watching this issue


              • Created: