[LUCENE-10423] Remove uses of wall-clock time in codebase

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Lucene Fields: New
• Assignee: Marios Trivyzas
• Reporter: Robert Muir
• Created: 16/Feb/22 10:22
• Updated: 28/Aug/22 16:36

Description

Followup to LUCENE-10421

Code in the library shouldn't rely on wall-clock time. If you look at all the places doing this, they are basically all bad news.

Most tests doing this are "iterating for some amount of wall-clock time" which causes them to instead just be non-reproducible. These should be changed to use a fixed number of loop iterations instead.

It would really be great to ban this stuff in forbidden apis. It is even in the configuration file, just currently commented out.

Issue Links

• relates to: LUCENE-10421 Non-deterministic results from KnnVectorQuery?