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

Robert Muir commented on LUCENE-6916:
-------------------------------------

I'm not sure we should do this, it makes test cases significantly harder to 
read, but at what benefit?

We should not be failing on file leaks if the test already fails.


> BaseDirectoryTestCase should use try-with-resources for its Directories
> -----------------------------------------------------------------------
>
>                 Key: LUCENE-6916
>                 URL: https://issues.apache.org/jira/browse/LUCENE-6916
>             Project: Lucene - Core
>          Issue Type: Improvement
>            Reporter: Alan Woodward
>            Assignee: Alan Woodward
>            Priority: Minor
>         Attachments: LUCENE-6916.patch
>
>
> I'm playing around with writing a nio2 FileSystem implementation for HDFS 
> that will work with Directory, and it currently leaks threads everywhere 
> because if a BaseDirectoryTestCase test fails it doesn't close its Directory. 
>  This obviously won't be a problem if everything passes, but it will probably 
> be a while before that's true and it makes iterative development a bit of a 
> pain.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to