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

MergePolicy.OneMerge.segments should be List<SegmentInfo> not SegmentInfos, Remove Vector<SI> subclassing from SegmentInfos & more refactoring

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 3.2, 4.0-ALPHA
    • None
    • None
    • New, Patch Available

    Description

      SegmentInfos carries a bunch of fields beyond the list of SI, but for merging purposes these fields are unused.

      We should cutover to List<SI> instead.

      Also SegmentInfos subclasses Vector<SI>, this should be removed and the collections be hidden inside the class. We can add unmodifiable views on it (asList(), asSet()).

      Attachments

        1. LUCENE-3084.patch
          12 kB
          Michael McCandless
        2. LUCENE-3084-3.x-only.patch
          34 kB
          Uwe Schindler
        3. LUCENE-3084-trunk-only.patch
          33 kB
          Uwe Schindler
        4. LUCENE-3084-trunk-only.patch
          33 kB
          Uwe Schindler
        5. LUCENE-3084-trunk-only.patch
          33 kB
          Uwe Schindler
        6. LUCENE-3084-trunk-only.patch
          31 kB
          Uwe Schindler
        7. LUCENE-3084-trunk-only.patch
          28 kB
          Uwe Schindler
        8. LUCENE-3084-trunk-only.patch
          24 kB
          Uwe Schindler
        9. LUCENE-3084-trunk-only.patch
          23 kB
          Uwe Schindler
        10. LUCENE-3084-trunk-only.patch
          7 kB
          Uwe Schindler
        11. LUCENE-3084-trunk-only.patch
          7 kB
          Uwe Schindler

        Activity

          People

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

            Dates

              Created:
              Updated:
              Resolved: