[HADOOP-15316] GenericTestUtils can exceed maxSleepTime

• Type: Bug
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Fix Version/s: 3.2.0
• Assignee: Adam Antal
• Reporter: Sean Mackrory
• Created: 14/Mar/18 20:10
• Updated: 12/Jul/18 16:00
• Resolved: 12/Jul/18 16:00

Description

Probably shouldn't ever cause an issue, especially since Thread.sleep() can cause longer delays beyond your control anyway, but for larger values this could still behave unpredicatably in practice.

Thread.sleep(r.nextInt(maxSleepTime) + minSleepTime);

should be

Thread.sleep(r.nextInt(maxSleepTime - minSleepTime) + minSleepTime)