dweiss opened a new pull request, #15327: URL: https://github.com/apache/lucene/pull/15327
This builds on top of #15325 so it'll be rebased after that is merged. LuceneTestCase.random() does have an overhead (needs to loop up randomized context, then returns an asserting Random instance to detect if Random is passed between threads, which shouldn't be done for reproducibility). This adds two build options to allow detecting mis-use of random() in intense loops. These are low-level settings, not meant to be used for devs in general but useful for detecting such abuse cases. ``` tests.random.maxacquires = [empty] # Max number of per-test calls to LuceneTestCase.random() tests.random.maxcalls = [empty] # Max number of calls to Randoms returned by LuceneTestCase.random() t ``` I've fixed some of the test cases to be more graceful. I don't see a lot of an impact on my machine but it's an improvement nonetheless. -- 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]
