[LUCENE-9455] ExitableTermsEnum (in ExitableDirectoryReader) should sample next()

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 8.8
• Component/s: core/other
• Labels: newdev
• Lucene Fields: New
• Assignee: Unassigned
• Reporter: David Smiley
• Created: 11/Aug/20 13:45
• Updated: 28/Aug/22 16:05
• Resolved: 27/Oct/20 08:57

Description

ExitableTermsEnum calls "checkAndThrow" on every call to next(). This is too expensive; it should sample. I observed ElasticSearch uses the same approach; I think Lucene would benefit from this:
https://github.com/elastic/elasticsearch/blob/4af4eb99e18fdaadac879b1223e986227dd2ee71/server/src/main/java/org/elasticsearch/search/internal/ExitableDirectoryReader.java#L151

CC jimczi

Issue Links

• links to: GitHub Pull Request #1998