javanna opened a new pull request, #16215: URL: https://github.com/apache/lucene/pull/16215
The identical 20-line block that verifies no live documents exist past the last collected doc in a leaf appeared four times (doSetNextReader + post-loop in both checkSkipTo and checkFirstSkipTo). Extract it into a private helper so the logic lives in one place. Also adds TestQueryUtils to verify that checkSkipTo and checkFirstSkipTo actually detect broken scorer behaviour (advance returning the wrong doc, score instability, score mismatch between advance and nextDoc paths). This is the first of a series of PRs aimed at removing the deprecated `search(Query, Collector)` usages in `QueryUtils`. -- 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]
