luyuncheng commented on PR #12237: URL: https://github.com/apache/lucene/pull/12237#issuecomment-1524835838
> Except, the `ensureValidMerge` would throw an exception in this case -- can we fix the duplicated code to throw that same exception? @mikemccand thanks for your advice, i added throw exception in https://github.com/apache/lucene/pull/12237/commits/d14f5d4a254b425d2b4e3e14e71b4510b9fec818 and fixed test `testLiveMaxMergeCount` when throw the same exception in iterator ``` MergePolicy selected a segment (_2e) that is not in the current index _0(10.0.0) ``` -- 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
