[
https://issues.apache.org/jira/browse/HADOOP-6276?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Allen Wittenauer resolved HADOOP-6276.
--------------------------------------
Resolution: Won't Fix
Closing this due to lack of consensus.
> test-patch script should -1 patches that increase the number of
> Thread.sleep() calls in tests
> ---------------------------------------------------------------------------------------------
>
> Key: HADOOP-6276
> URL: https://issues.apache.org/jira/browse/HADOOP-6276
> Project: Hadoop Common
> Issue Type: Improvement
> Components: test
> Reporter: Nigel Daley
> Assignee: Giridharan Kesavan
>
> Calls to Thread.sleep() in tests are likely dubious. There are rare
> occasions where these are required, but in general they lead to flaky unit
> tests that fail sporadically. We used to be much better at code reviewing
> these out of patches, but have lost ground on that in the last few years (and
> our flaky unit tests show that!).
> I propose we have the test-patch script ensure that a patch does not add to
> the number of Thread.sleep() calls in *tests* by giving a -1 to patches that
> do increase it. I suggest this is a simple grep for lines containing
> "Thread.sleep(" and a "+" character in the first column.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
