jpountz commented on PR #14922: URL: https://github.com/apache/lucene/pull/14922#issuecomment-3067963440
Yes, something like that. To properly meet the contract, I'd rather do `advanceWithinBlock(destTo)` first, and then fail if the current doc ID is less than upTo. This way, the current doc ID would be in [destTo, upTo) after the exception is thrown, as would be the case with the default implementation whose behavior we need to mimic. -- 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
