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

CheckIndex is dog slow when checking term vectors

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 5.1, 6.0
    • None
    • None
    • New

    Description

      I'm working on a test that creates a biggish index and I noticed the CheckIndex takes a surprisingly long time to check term vectors.

      I profiled it and uncovered that we spend a lot of time (not sure this explains all of it) in Terms.getMin/getMax. Since CompressingTermVectorsReader doesn't impl these methods efficiently (which is fine), we fallback to super's impl, which does a digit-by-digit binary search using seekCeil.

      But for TVs this sometimes results in a linear scan.

      I think CheckIndex should not check Terms.getMin/Max for TVs?

      Attachments

        1. LUCENE-6233.patch
          13 kB
          Michael McCandless
        2. LUCENE-6233.patch
          14 kB
          Robert Muir
        3. LUCENE-6223.patch
          33 kB
          Robert Muir

        Activity

          People

            mikemccand Michael McCandless
            mikemccand Michael McCandless
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: