gsmiller commented on pull request #597: URL: https://github.com/apache/lucene/pull/597#issuecomment-1011231123
I'm going to run some more local benchmarks with some of these other facet-related changes reverted. While I agree that none of them seem like they should impact the `#countAll` performance, it also doesn't make sense to me that LUCENE-10372 would, so I'm just going to dig into all of them. I'll post my benchmark results here. If that still doesn't turn anything interesting up, I'll go ahead and merge this revert to see the impact on the nightly run. -- 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]
