[LUCENE-10050] Remove DrillSideways#search(DrillDownQuery,Collector) in favor of DrillSideways#search(DrillDownQuery,CollectorManager)

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 9.1, 10.0 (main)
• Component/s: modules/facet
• Lucene Fields: New
• Assignee: Unassigned
• Reporter: Greg Miller
• Created: 14/Aug/21 01:30
• Updated: 28/Aug/22 16:24
• Resolved: 05/Feb/22 00:15

Description

With similar motivation to LUCENE-10002, we should consider doing away with the ability to directly provide a Collector to DrillSideways in favor of always accepting a CollectorManager. Just like with IndexSearcher, it's trappy that you can provide an Executor when setting up DrillSideways and then not leverage it by directly providing a single Collector.

Issue Links

• links to: GitHub Pull Request #632
• links to: GitHub Pull Request #650