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

MockDirectoryWrapper should wrap the lockfactory

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 3.3, 4.0-ALPHA
    • general/test
    • None
    • New

    Description

      After applying the patch from LUCENE-3147, I added a line to make the test fail if it cannot remove its temporary directory.

      I ran 'ant test' on linux 50 times, and it passed all 50 times.
      But on windows, it failed often because of write.lock... this is because of unclosed writers in the test.

      MockDirectoryWrapper is currently unaware of this write.lock, I think it should wrap the lockfactory so that .close() will fail if there are any outstanding locks.
      Then hopefully these tests would fail on linux too.

      Attachments

        1. LUCENE-3152.patch
          9 kB
          Robert Muir

        Issue Links

          Activity

            People

              Unassigned Unassigned
              rcmuir Robert Muir
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: