[
https://issues.apache.org/jira/browse/YARN-6217?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15930346#comment-15930346
]
Jason Lowe commented on YARN-6217:
----------------------------------
I tend to agree. Originally there was an edict to put a timeout on each test
because the build wasn't doing a good job of handling tests that timed out.
However that has since been fixed. An explicit test timeout still makes a lot
of sense when the test has a good chance of deadlocking when broken (e.g.:
needing to carefully synchronize a number of threads, wait for barriers, doing
a polling loop, etc.), but I don't think that's the case with the tests here.
> TestLocalCacheDirectoryManager test timeout is too aggressive
> -------------------------------------------------------------
>
> Key: YARN-6217
> URL: https://issues.apache.org/jira/browse/YARN-6217
> Project: Hadoop YARN
> Issue Type: Bug
> Components: test
> Reporter: Jason Lowe
> Assignee: Miklos Szegedi
> Attachments: YARN-6217.000.patch
>
>
> TestLocalCacheDirectoryManager#testDirectoryStateChangeFromFullToNonFull has
> only a one second timeout. If the test machine hits an I/O hiccup it can
> fail. The test timeout is too aggressive, and I question whether this test
> even needs an explicit timeout specified.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
