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

Merging multiple indexes does not maintain document order.

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.9
    • Fix Version/s: 2.0.0
    • Component/s: None
    • Labels:
      None
    • Environment:

      Windows XP, Java 1.5.0_05
      Fedora Core 3 - 64 bit - Java 1.5.0_06 (64 bit)
      Lucene 1.9.1 (issue tracker doesn't have a 1.9.1 option)

      Description

      When I merge multiple indexes into a single, empty index, the document addition order is not being maintained.

      Self contained test case coming (as soon as I figure out how to attach it)

      1. LuceneBug.java
        4 kB
        Dan Armbrust
      2. TestMergeIndexDocOrder.java
        4 kB
        Hoss Man

        Activity

        Hide
        armbrust Dan Armbrust added a comment -

        The attached JUnit test case should demonstrate the problem. See the comments inside of the test case.

        Show
        armbrust Dan Armbrust added a comment - The attached JUnit test case should demonstrate the problem. See the comments inside of the test case.
        Hide
        hossman Hoss Man added a comment -

        I refactored the previous test to demonstrate the problem both with FSDirectory and with RAMDirectory.

        interestingly: they fail at differnet document numbers.

        file should live in package org.apache.lucene.index

        Show
        hossman Hoss Man added a comment - I refactored the previous test to demonstrate the problem both with FSDirectory and with RAMDirectory. interestingly: they fail at differnet document numbers. file should live in package org.apache.lucene.index
        Hide
        yseeley@gmail.com Yonik Seeley added a comment -

        Fix committed to the latest dev version (soon to be Lucene 2.0)

        Show
        yseeley@gmail.com Yonik Seeley added a comment - Fix committed to the latest dev version (soon to be Lucene 2.0)
        Hide
        armbrust Dan Armbrust added a comment -

        Fix verified here.

        Thanks for the rapid repair.

        Show
        armbrust Dan Armbrust added a comment - Fix verified here. Thanks for the rapid repair.

          People

          • Assignee:
            yseeley@gmail.com Yonik Seeley
            Reporter:
            armbrust Dan Armbrust
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development