costin commented on PR #16142: URL: https://github.com/apache/lucene/pull/16142#issuecomment-4576523845
Thanks @jainankitk Re 1: this is what test "testDocIDRunEndOnlyUsesLeadIterators" intends to cover: the shorter/lower-cost clause is placed in the lead heap, while the longer/higher-cost clause is placed in `otherIterators`; both are positioned on doc `100`, and the expected run end is `110` rather than `200`. I've renamed the method to make it more clear along with some comments. Re 2: I've expanded the docIDRunEnd() comment. -- 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]
