shatejas commented on PR #13985: URL: https://github.com/apache/lucene/pull/13985#issuecomment-2492118585
> I broke the tests again, by adding more asserts, as I'm a little uncomfortable with the finishMergeCount <= 0. Now I broke something else. I'll get to fixing this, or backout the extra asserts. For reference this reproduces: Took a look, The mismatch between `mergeInstanceCount` and `mergeInstance` is because mergeInstanceCount is being updated in parent and mergeInstance is updated to true during getMergeInstance(). If search is happening during merge there is a good chance that it runs into a scenario where `mergeInstanceCount = 1` but `mergeInstance` is false. Considering the test has commit() call followed by search, I think its running into the scenario -- 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
