fthevenet commented on PR #12212: URL: https://github.com/apache/lucene/pull/12212#issuecomment-1488527286
OK, so I dug a little deeper in what happens in the failing instance of the random test. It fails because of an ÌllegalArgumentException` thrown by `TopFieldCollector::populateScores` at line 552 https://github.com/apache/lucene/blob/b84b360f588c60e2ebdcaa2ce2e6d49f99dd9deb/lucene/core/src/java/org/apache/lucene/search/TopFieldCollector.java#L526-L556 But the reason it does not fail before my fix is that in this case the search returns no docs in the first place (which was the tell sign of the underlying issue that got me started on it) and therefore never goes into the code path that throws. So at this stage, I can't tell if the fix is incorrect and the root cause for this particular failure or if it is just surfacing another problem somewhere else, by virtue of having fixed stuff... -- 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
