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

Michael McCandless commented on LUCENE-6999:
--------------------------------------------

This only affect trunk.

> Lucene60PointsWriter has a buggy close method
> ---------------------------------------------
>
>                 Key: LUCENE-6999
>                 URL: https://issues.apache.org/jira/browse/LUCENE-6999
>             Project: Lucene - Core
>          Issue Type: Bug
>            Reporter: Robert Muir
>            Assignee: Michael McCandless
>             Fix For: Trunk, 6.0
>
>         Attachments: LUCENE-6999.patch, LUCENE-6999_test.patch
>
>
> This will leak a file handle on e.g. disk full or other exceptions
> {noformat}
>   @Override
>   public void close() throws IOException {
>     if (closed == false) {
>       CodecUtil.writeFooter(dataOut); // something bad happens
>       dataOut.close();
> {noformat}
> I'm not a fan of these complex close methods, should we add a finish() to the 
> codec api or similar?



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

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

Reply via email to