jpountz commented on PR #15023: URL: https://github.com/apache/lucene/pull/15023#issuecomment-3144623188
Very cool. Have you been able to measure any speedup with this approach? FYI, this breaks some API contracts, e.g. a `BulkScorer` is expected to score ranges of doc IDs in doc ID order. You would need to create a new BulkScorer every time that you need to go back. Somewhat related, I think that implementing this via a helper function that evaluates a query against an entire index would be better than a query wrapper, you'd be less subjet to fewer expectations. E.g. `IndexSearcher` sometimes splits the doc ID space into ranges of increasing size via `TimeLimitingBulkScorer`. This would likely not play well with this change which really expects to score the whole index at once. This would also allow you to order ranges by priority across all segments and not only within a single segment. -- 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-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
