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

SearcherManager and NRTManager should be in the same package

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 3.5, 4.0-ALPHA
    • None
    • None
    • 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

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

            Dates

              Created:
              Updated:
              Resolved: