romseygeek commented on PR #15436: URL: https://github.com/apache/lucene/pull/15436#issuecomment-3755881746
> For less concurrent searching, the optimization would be more impactful if the list of concurrent search tasks was also sorted before being submitted to the TaskExecutor probably It turns out that this change doesn't really make any difference at all for concurrent searching, because we have no way of communicating sort values that aren't scores between different Tasks. I think that's definitely worth investigating in a follow-up (we'd need to change the `getSlices()` API signature to pass Sorts around for a start). -- 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]
