jpountz commented on issue #14986: URL: https://github.com/apache/lucene/issues/14986#issuecomment-3117890103
> Is there a good reason to not have this ring buffer tracking at a per segment level? To your point, there are many segments, so this would increase heap usage. Furthermore, the typical case is that segments need to process the same query as the top-level, so this wouldn't help much in other cases than the one that your are describing. Passing these per-segment rewrites through the cache feels wrong. The cache should never see these per-segment rewrites, only the `TermInSetQuery`? I believe that we could fix it by calling `Query#createWeight` rather than `IndexSearcher#createWeight` in `AbstractMultiTermQueryConstantScoreWrapper`. -- 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
