Re: [PR] Compute multiple float aggregations in one go [lucene]

["gsmiller (via GitHub)" <gi...@apache.org>]

gsmiller commented on PR #12547:
URL: https://github.com/apache/lucene/pull/12547#issuecomment-2379648625

   @stefanvodita do you think this change is still worth moving forward after the new sandbox faceting implementation was added? Being able to compute an arbitrary number of aggregations in one pass is part of what that implementation was designed to do, so I'm not sure if it also makes sense to add this into the existing module? Curious what you think.


-- 
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: issues-unsubscribe@lucene.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org