jpountz commented on PR #15058: URL: https://github.com/apache/lucene/pull/15058#issuecomment-3194637811
I don't think we're thinking of `DocIdSetIterator#cost` as something that is expected to get updated during query evaluation. Some implementations may accidentally do this through delegation (e.g. competitive iterators) but I'm a bit wary about relying on this. I'd rather do something specific for competitive iterators, e.g. evaluating it last in the beginning (or wherever its cost would put it), and moving it to the front when it starts looking very sparse, e.g. by looking at how many docs it skips at once (rather than looking at the cost). -- 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
