From 4f3f9391b035d7f7e285c332770c6c1ede9a5a85 Mon Sep 17 00:00:00 2001 From: Sean Mackrory Date: Thu, 12 Jul 2018 16:45:07 +0200 Subject: [PATCH] HADOOP-15316. GenericTestUtils can exceed maxSleepTime. Contributed by Adam Antal. --- .../src/test/java/org/apache/hadoop/test/GenericTestUtils.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/test/GenericTestUtils.java b/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/test/GenericTestUtils.java index 3e9da1b45fd..01128940014 100644 --- a/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/test/GenericTestUtils.java +++ b/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/test/GenericTestUtils.java @@ -661,7 +661,7 @@ public SleepAnswer(int minSleepTime, int maxSleepTime) { public Object answer(InvocationOnMock invocation) throws Throwable { boolean interrupted = false; try { - Thread.sleep(r.nextInt(maxSleepTime) + minSleepTime); + Thread.sleep(r.nextInt(maxSleepTime - minSleepTime) + minSleepTime); } catch (InterruptedException ie) { interrupted = true; }