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

Steve Loughran commented on HADOOP-14247:
-----------------------------------------

I'd like the assert check pulled from teardown. Otherwise the sequence of: test 
failure, teardown failure, will only show the teardown exception, not the 
underlying test failure. I'd end up having to edit the code, comment out the 
assert and rerun. Catch and log, fine: but teardowns mustn't raise exceptions

> FileContextMainOperationsBaseTest should clean up test root path
> ----------------------------------------------------------------
>
>                 Key: HADOOP-14247
>                 URL: https://issues.apache.org/jira/browse/HADOOP-14247
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: fs, test
>    Affects Versions: 2.8.0
>            Reporter: Mingliang Liu
>            Assignee: Mingliang Liu
>            Priority: Minor
>         Attachments: HADOOP-14247.000.patch
>
>
> {{fileContextTestHelper}} will create random test root directory for each 
> test case, say $RANDOM. After test, we only delete the $RANDOM/test directory 
> in tearDown(). This will leave dangling directories under the {{test-dir}} 
> directory. For S3A with S3Guard enabled, those useless test directories will 
> stay in DynamoDB forever. Let's clear out the test root path, which is 
> (generated randomly and) safe to remove.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

Reply via email to