epotyom commented on PR #15518: URL: https://github.com/apache/lucene/pull/15518#issuecomment-3690051992
> +1, thanks @epotyom — this adds the capabilities of the first faceting impl (two-phase) to the new sandbox faceting engine? You are correct. > > when all matches must be known before computing facets > > >What's an example use-case? In Amazon product search, we want to add support for rank-dependent associative facets, e.g. sorting facets by max doc relevance, where the relevance expression depends on the ranks of the top-N matching documents. Other potential use cases that come to mind are [dynamic ranges](https://github.com/apache/lucene/blob/247b96e7f3ffb23de140718449c1b3ab9082b944/lucene/demo/src/java/org/apache/lucene/demo/facet/DynamicRangeFacetsExample.java) (which require numeric values for all matches before they can be computed), and percentile aggregates for facets. Note that this change isn’t the only thing we need to integrate these features with the new faceting API, but it enables them in the future. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
