[LUCENE-10614] Properly support getTopChildren in RangeFacetCounts

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 10.0 (main)
• Fix Version/s: 10.0 (main)
• Component/s: modules/facet
• Lucene Fields: New
• Assignee: Unassigned
• Reporter: Greg Miller
• Created: 13/Jun/22 23:51
• Updated: 28/Aug/22 16:42
• Resolved: 11/Jul/22 16:19

Description

As mentioned in LUCENE-10538, RangeFacetCounts is not implementing getTopChildren. Instead of returning "top" ranges, it returns all user-provided ranges in the order the user specified them when instantiating. This is probably more useful functionality, but it would be nice to support getTopChildren as well.

LUCENE-10550 is introducing the concept of getAllChildren, so once that lands, we can replace the current implementation of getTopChildren with an actual "top children" implementation and direct users to getAllChildren if they want to maintain the current behavior.

Issue Links

• links to: GitHub Pull Request #974