spinscale opened a new pull request, #15725: URL: https://github.com/apache/lucene/pull/15725
The `LuceneTestCase` has quite a few methods that allow passing a `Random` parameter. In a few of those methods that passed parameter is not used, but the standard `random()` supplier is called - or usages mixed between the argument and the standard `random()` call. While this change seems useful to me from a code perspective, it might also mean, that trying to repeat currently failing test runs might pass with this. Please just go ahead and close if you think it's not worth it. I also renamed the `configureRandom` method, because it was not clear to me what it's doing. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
