[LUCENE-5519] Make queueDepth enforcing optional in TopNSearcher

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 4.7
• Fix Version/s: 4.8, 6.0
• Component/s: core/FSTs
• Lucene Fields: New, Patch Available
• Assignee: Unassigned
• Reporter: Simon Willnauer
• Created: 11/Mar/14 20:49
• Updated: 28/Aug/22 14:02
• Resolved: 12/Mar/14 20:31

Description

currently TopNSearcher enforces the maxQueueSize based on rejectedCount + topN. I have a usecase where I just simply don't know the exact limit and I am ok with a top N that is not 100% exact. Yet, if I don't specify the right upper limit for the queue size I get an assertion error when I run tests but the only workaround it to make the queue unbounded which looks odd while it would possibly work just fine. I think it's fair to add an option that just doesn't enforce the limit and if it shoudl be enforced we throw a real exception.