benwtrent commented on PR #13475: URL: https://github.com/apache/lucene/pull/13475#issuecomment-2159258542
I ran some more tests. My logic for "move assertion to when we first read the index segment values" still doesn't work. Its possible with index sorting that we read doc values once from the merge thread, and then later kick off doc value merging in a separate thread. While this isn't executed at the same time, it can trip assertions. I am not sure how to avoid this or better protect this code. -- 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]
