benwtrent commented on PR #15556: URL: https://github.com/apache/lucene/pull/15556#issuecomment-3745769656
Let me try and be clearer than before. I am very confused why we need a new concept that leaks outside of Directories and IndexInput at all. We should not have a new primary API item that is foisted upon all data formats individually. Why isn't this something that is handled in some indexinput wrapper and a directory wrapper? Prefetch is allowed to be a noop, so the wrapper can prefetch if desired or not. If the concern is because MMAP directory does this little "hey, are these pages loaded?" check, the solution isn't a new API concept that is then satisfiable everywhere. -- 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]
