romseygeek commented on PR #15729: URL: https://github.com/apache/lucene/pull/15729#issuecomment-3935567067
> On digging deeper, I noticed that in places like DocValuesSkipper, we rely on globalMinValue / globalMaxValue, which internally call getSkipper(), and prefetching doesn’t seem feasible there. For those cases, maybe we could expose a lightweight variant of the getSkipper() implementation that avoids prefetch? Prefetch is a pretty low-level API that I don't think we should be exposing to consumers of DocValuesSkipper. And a range query being able to quickly short-cut because it doesn't match any documents in a segment is pretty common for things like logging usecases where things are sorted by timestamp, so I think this is a useful change. -- 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]
