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

SearcherManager and NRTManager should be in the same package

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.5, 4.0-ALPHA
    • Component/s: None
    • Labels:
      None
    • Lucene Fields:
      New, Patch Available

      Description

      I didnt even know NRTManager was still around, because its in the .index package, whereas SearcherManager is in the .search package.

      Separately, I don't like that this stuff is so 'hard' with core lucene... would it be so bad if this stuff was added to core?

      I suspect a lot of people have issues with this stuff (see http://www.lucidimagination.com/search/document/37964e5f0e5d733b) for example.

      Worst case is just that, combine mistakes with trying to manage this stuff with MMap unmapping and total lack of error detection
      for searching closed readers (LUCENE-3439) and its a mess.

        Attachments

        1. LUCENE-3558.patch
          88 kB
          Simon Willnauer
        2. LUCENE-3558.patch
          71 kB
          Simon Willnauer
        3. LUCENE-3558.patch
          70 kB
          Simon Willnauer
        4. LUCENE-3558.patch
          59 kB
          Simon Willnauer
        5. LUCENE-3558.sh
          0.9 kB
          Simon Willnauer
        6. LUCENE-3558.sh
          0.7 kB
          Simon Willnauer
        7. LUCENE-3558.sh
          0.6 kB
          Simon Willnauer

          Activity

            People

            • Assignee:
              simonw Simon Willnauer
              Reporter:
              rcmuir Robert Muir
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: