leventov opened a new issue #6227: Usage of `new Random(constant seed)` in tests URL: https://github.com/apache/incubator-druid/issues/6227 On the one hand, it's good because it makes the tests 100% deterministic and test failures reproducible. On the other hand, compared to `ThreadLocalRandom.current()` or `new Random(seed)` with a _random_ seed, a constant seed may not allow to explore all input conditions and discover some failing corner cases. So ideally the seed should be generated randomly and be a *parameter* of a test, recorded in the test name. So when a test fails, it could be run locally with the known seed parameter in order to reproduce a test failure.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
