[ 
https://issues.apache.org/jira/browse/DERBY-3889?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12639005#action_12639005
 ] 

Kristian Waagan commented on DERBY-3889:
----------------------------------------

The patch looks good to me. The refactoring makes it easer to write correct 
code.
+1 to commit

I reviewed the code and ran the repro. Without the patch I observed thousands 
of temporary files, and with the patch the ls command showed zero or one file 
in the tmp directory. I have also started the regression tests.

> LOBStreamControl.truncate() doesn't delete temporary files
> ----------------------------------------------------------
>
>                 Key: DERBY-3889
>                 URL: https://issues.apache.org/jira/browse/DERBY-3889
>             Project: Derby
>          Issue Type: Bug
>          Components: JDBC
>    Affects Versions: 10.3.1.4, 10.4.2.0
>            Reporter: Knut Anders Hatlen
>            Assignee: Knut Anders Hatlen
>         Attachments: derby-3889-1a.diff, derby-3889-1a.stat, TruncateClob.java
>
>
> If LOBStreamControl.truncate() reduces the size of the lob so that it fits in 
> the internal buffer, it closes the temporary file, but it doesn't remove it. 
> See the attached Java class (TruncateClob.java) which fills the tmp directory 
> with temporary lob files, even if each lob is explicitly freed.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to