lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael McCandless (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (LUCENE-6835) Directory.deleteFile should "own" retrying deletions on Windows
Date Thu, 04 Feb 2016 14:27:39 GMT

    [ https://issues.apache.org/jira/browse/LUCENE-6835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15132385#comment-15132385
] 

Michael McCandless commented on LUCENE-6835:
--------------------------------------------

bq. Should we add a TODO to IndexFileDeleter to try to remove the LUCENE-6684 hack later too?

Good catch, I'll put a TODO.  I wanted to just remove it now :)  But then that test would
fail again, because FSDirectory makes no effort to sugarcoat a false NSFE...

I'll add sort to {{listAll}} across the board and add a test case here ...

> Directory.deleteFile should "own" retrying deletions on Windows
> ---------------------------------------------------------------
>
>                 Key: LUCENE-6835
>                 URL: https://issues.apache.org/jira/browse/LUCENE-6835
>             Project: Lucene - Core
>          Issue Type: Improvement
>            Reporter: Michael McCandless
>             Fix For: 5.5, master
>
>         Attachments: LUCENE-6835.patch, LUCENE-6835.patch
>
>
> 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 ...



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org


Mime
View raw message