[Hadoop-common-dev] [jira] Created: (HADOOP-5483) Directory/file cleanup thread throws IllegalStateException

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

Tsz Wo (Nicholas), SZE updated HADOOP-5483:
-------------------------------------------

Component/s: (was: dfs)
mapred

HADOOP-5285 was committed to 0.19. Not sure whether this should be a blocker.

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

Devaraj Das updated HADOOP-5483:
--------------------------------

Attachment: 5483.1.patch

Here is a straightforward patch, where the cleanup thread catches Exception and ignores it.

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

Devaraj Das updated HADOOP-5483:
--------------------------------

Fix Version/s: 0.20.0
Affects Version/s: 0.20.0
Status: Patch Available (was: Open)

[ https://issues.apache.org/jira/browse/HADOOP-5483?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12682165#action_12682165 ]

Hadoop QA commented on HADOOP-5483:
-----------------------------------

-1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12402217/5483.1.patch
against trunk revision 754645.

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

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

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

+1 javac. The applied patch does not increase the total number of javac compiler warnings.

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

+1 Eclipse classpath. The patch retains Eclipse classpath integrity.

+1 release audit. The applied patch does not increase the total number of release audit warnings.

-1 core tests. The patch failed core unit tests.

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

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

This message is automatically generated.

[ https://issues.apache.org/jira/browse/HADOOP-5483?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12682221#action_12682221 ]

Amareshwari Sriramadasu commented on HADOOP-5483:
-------------------------------------------------

Patch looks fine

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

Devaraj Das updated HADOOP-5483:
--------------------------------

Resolution: Fixed
Hadoop Flags: [Reviewed]
Status: Resolved (was: Patch Available)

I just committed this.

[ https://issues.apache.org/jira/browse/HADOOP-5483?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12683091#action_12683091 ]

Hudson commented on HADOOP-5483:
--------------------------------

Integrated in Hadoop-trunk #783 (See [http://hudson.zones.apache.org/hudson/job/Hadoop-trunk/783/])