[
https://issues.apache.org/jira/browse/LUCENE-4308?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13435454#comment-13435454
]
Robert Muir commented on LUCENE-4308:
-------------------------------------
{quote}
The behavior here is correct (two exceptions) because the second exception is a
message indicating the temporary folder (for the unclosed directory) could not
be removed. Not being able to remove a temporary folder is a bad thing
especially on windows where this may affect things down the execution chain.
{quote}
I (strongly) don't agree with this. If a test failed, we shouldnt be deleting
its index directory.
If we arent going to change this, then ill open an issue to never delete
directories at all, because this is really bad and I've been frustrated with it
for a while.
I already have to *edit every test* and change it to make indexes in
/tmp/myIndex if a test fails so I can debug it: this is really horrible.
Secondly, we cannot remove the files if we cannot remove the Directory,
otherwise *ridiculous* exceptions from other threads are going to happen,
absurdly confusing ones that could send people off into hours of debugging that
they shouldnt be doing.
> Check what's wrong with reporting unclosed directories with previous errors
> ---------------------------------------------------------------------------
>
> Key: LUCENE-4308
> URL: https://issues.apache.org/jira/browse/LUCENE-4308
> Project: Lucene - Core
> Issue Type: Bug
> Reporter: Dawid Weiss
> Assignee: Dawid Weiss
> Attachments: LUCENE-4308.patch
>
>
> {code}
> -Dtests.seed=75DBD696A5D72F04
> -Dtestcase=TestFailIfDirectoryNotClosed
> {code}
> Windows.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
