Issue Details (XML | Word | Printable)

Key: HADOOP-2844
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Tsz Wo (Nicholas), SZE
Reporter: Tsz Wo (Nicholas), SZE
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Hadoop Common

A SequenceFile.Reader object is not closed properly in CopyFiles

Created: 16/Feb/08 12:01 AM   Updated: 21/May/08 08:05 PM
Return to search
Component/s: util
Affects Version/s: None
Fix Version/s: 0.17.0

Time Tracking:
Not Specified

File Attachments:
  Size
Text File Licensed for inclusion in ASF works 2844_20080215.patch 2008-02-16 12:02 AM Tsz Wo (Nicholas), SZE 2 kB
Issue Links:
Reference
 

Resolution Date: 19/Feb/08 05:58 AM


 Description  « Hide
In CopyFiles line 186, the SequenceFile.Reader sl is never closed.

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Mahadev konar added a comment - 16/Feb/08 12:45 AM
+1 .. reviewed.

Tsz Wo (Nicholas), SZE added a comment - 16/Feb/08 12:48 AM
This issue fixes the test problem reported in HADOOP-2813. Since this patch fixes a test problem, no additional test is included.

Hadoop QA added a comment - 16/Feb/08 01:50 AM
-1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12375731/2844_20080215.patch
against trunk revision 619744.

@author +1. The patch does not contain any @author tags.

tests included -1. The patch doesn't appear to include any new or modified tests.
Please justify why no tests are needed for this patch.

javadoc +1. The javadoc tool did not generate any warning messages.

javac +1. The applied patch does not generate any new javac compiler warnings.

release audit +1. The applied patch does not generate any new release audit warnings.

findbugs +1. The patch does not introduce any new Findbugs warnings.

core tests +1. The patch passed core unit tests.

contrib tests +1. The patch passed contrib unit tests.

Test results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/1809/testReport/
Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/1809/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/1809/artifact/trunk/build/test/checkstyle-errors.html
Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch/1809/console

This message is automatically generated.


dhruba borthakur added a comment - 19/Feb/08 05:58 AM
I just committed this. Thanks Nicholas!

Hudson added a comment - 19/Feb/08 12:08 PM