[ 
https://issues.apache.org/jira/browse/HADOOP-9681?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Chris Nauroth updated HADOOP-9681:
----------------------------------

    Hadoop Flags: Reviewed

+1 for the patch.  Thanks for incorporating the feedback.  I verified that the 
tests pass on both Mac and Windows for both branches.  I'll commit this.
                
> FileUtil.unTarUsingJava() should close the InputStream upon finishing
> ---------------------------------------------------------------------
>
>                 Key: HADOOP-9681
>                 URL: https://issues.apache.org/jira/browse/HADOOP-9681
>             Project: Hadoop Common
>          Issue Type: Bug
>    Affects Versions: 3.0.0, 1-win, 2.1.0-beta
>            Reporter: Chuan Liu
>            Assignee: Chuan Liu
>            Priority: Minor
>         Attachments: HADOOP-9681-branch-1-win.patch, 
> HADOOP-9681-trunk.2.patch, HADOOP-9681-trunk.patch
>
>
> In {{FileUtil.unTarUsingJava()}} method, we did not close input steams 
> explicitly upon finish. This could lead to a file handle leak on Windows.
> I discovered this when investigating the unit test case failure of 
> {{TestFSDownload.testDownloadArchive()}}. FSDownload class will use 
> {{FileUtil.unTarUsingJava()}} to unpack some temporary archive file. Later, 
> the temporary file should be deleted. Because of the file handle leak, the 
> {{File.delete()}} method fails. The test case then fails because it assert 
> the temporary file should not exist.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to