benwtrent commented on PR #13475: URL: https://github.com/apache/lucene/pull/13475#issuecomment-2173140212
> Would it address the problem if we moved pulling these readers to the thread that actually does the merging? @jpountz I am not sure, the idea seems OK to me, but I have two concerns. If we are sorted by a doc_values field, we need to read the values to build the sort values. So, I am not sure how we can ever do that in the same thread the doc_values are merged by because all other threads need to know the sorting. When merging norms & doc values, they cause weird thrashing in `SortingCodecReader#getOrCreate` explained further in: https://github.com/apache/lucene/issues/13478#issuecomment-2160815181. -- 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
