[LUCENE-6835] Directory.deleteFile should "own" retrying deletions on Windows

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 6.0
• Lucene Fields: New
• Assignee: Unassigned
• Reporter: Michael McCandless
• Created: 12/Oct/15 10:32
• Updated: 28/Aug/22 14:44
• Resolved: 06/Feb/16 13:27

Description

Rob's idea:

Today, we have hairy logic in IndexFileDeleter to deal with Windows file systems that cannot delete still open files.

And with LUCENE-6829, where OfflineSorter now must deal with the situation too ... I worked around it by fixing all tests to disable the virus checker.

I think it makes more sense to push this "platform specific problem" lower in the stack, into Directory? I.e., its deleteFile method would catch the access denied, and then retry the deletion later. Then we could re-enable virus checker on all these tests, simplify IndexFileDeleter, etc.

Maybe in the future we could further push this down, into WindowsDirectory, and fix FSDirectory.open to return WindowsDirectory on windows ...

Issue Links

• is blocked by: LUCENE-6848 Add VirusCheckingFS mock filesystem