[LUCENE-10653] Should BlockMaxMaxscoreScorer rebuild its heap in bulk?

• Type: Improvement
• Status: Reopened
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 9.3
• Component/s: core/search
• Lucene Fields: New
• Assignee: Greg Miller
• Reporter: Greg Miller
• Created: 11/Jul/22 21:10
• Updated: 10/Oct/24 00:46
• Resolved: 19/Jul/22 22:46

Description

BMMScorer has to frequently rebuild its heap, and does do by clearing and then iteratively calling add. It would be more efficient to heapify in bulk. This is more academic than anything right now though since BMMScorer is only used with two-clause disjunctions, so it's sort of a silly optimization if it's not supporting a greater number of clauses.

Issue Links

• links to: GitHub Pull Request #1022