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

addIndexesNoOptimize should not enforce maxMergeDocs/maxMergeSize limit

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: 2.1, 2.2, 2.3
    • Fix Version/s: 2.3.2, 2.4
    • Component/s: core/index
    • Labels:
      None
    • Lucene Fields:
      New

      Description

      If you pass an index that has a segment > maxMergeDocs or maxMergeSize
      to addIndexesNoOptimize, it throws an IllegalArgumentException.

      But this check isn't reasonable because segment merging can easily
      produce segments over these sizes since those limits apply to each
      segment being merged, not to the final size of the segment produced.

      So if you set maxMergeDocs to X, build up and index, then try to add
      that index to another index that also has maxMergeDocs X, you can
      easily hit the exception.

      I think it's being too pedantic; I plan to just remove the checks for
      sizes.

        Attachments

        1. LUCENE-1254.patch
          2 kB
          Michael McCandless

          Activity

            People

            • Assignee:
              mikemccand Michael McCandless
              Reporter:
              mikemccand Michael McCandless
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: