sgup432 commented on PR #15729: URL: https://github.com/apache/lucene/pull/15729#issuecomment-3932253539
Prefetch doesn’t help when a range query doesn’t match any documents in a segment, but I believe those cases are relatively rare. Given that prefetch still seems like a reasonable approach and a worthwhile tradeoff to me. Would be good to run some benchmark to verify this change. 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? -- 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]
