jpountz commented on PR #14705: URL: https://github.com/apache/lucene/pull/14705#issuecomment-2944121468
For reference, there is an interesting PR at https://github.com/apache/lucene/pull/14714 that replaces `PriorityQueue` with a `LongHeap` when computing top-k hits by score. Results suggest that the `PriorityQueue` is not a bottleneck at all when k=100, but is a bottleneck for our fastest queries when k=1000. This PR doesn't change computing top-k hits by field, so tasks `TermMonthSort`, `TermTitleSort`, `TermDayOfYearSort` or `TermDTSort` would still use a `PriorityQueue` to track top hits. -- 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
