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

fix or deprecate TermsEnum.skipTo

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 2.9
    • core/index
    • None
    • New

    Description

      This method is a trap: it looks legitimate but it has hideously poor performance (simple linear scan implemented in the TermsEnum base class since none of the concrete impls override it with a more efficient implementation).

      The least we should do for 2.9 is deprecate the method with a strong warning about its performance.

      See here for background: http://www.lucidimagination.com/search/document/77dc4f8e893d3cf3/possible_terminfosreader_speedup

      And, here for historical context:

      http://www.lucidimagination.com/search/document/88f1b95b404ebf16/remove_termenum_skipto_term_target

      Attachments

        Activity

          People

            uschindler Uwe Schindler
            mikemccand Michael McCandless
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: