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

Pass potent SR to IRWarmer.warm(), and also call warm() for new segments

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 2.9.3, 3.0.2, 3.1, 4.0-ALPHA
    • core/index
    • None
    • New

    Description

      Currently warm() receives a SegmentReader without terms index and docstores.
      It would be arguably more useful for the app to receive a fully loaded reader, so it can actually fire up some caches. If the warmer is undefined on IW, we probably leave things as they are.

      It is also arguably more concise and clear to call warm() on all newly created segments, so there is a single point of warming readers in NRT context, and every subreader coming from getReader is guaranteed to be warmed up -> you don't have to introduce even more mess in your code by rechecking it.

      Attachments

        1. LUCENE-2311.patch
          5 kB
          Michael McCandless
        2. LUCENE-2311.patch
          3 kB
          Michael McCandless

        Activity

          People

            mikemccand Michael McCandless
            earwin Earwin Burrfoot
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: