jpountz commented on PR #13196: URL: https://github.com/apache/lucene/pull/13196#issuecomment-2012967809
I remember this kind of things being discussed more than 10 years ago, it's extremely exciting to see it close to being included in the default `Directory`! > current "readOnce => MADV_SEQUENTIAL" is just an example as this is the main issue: We merge segments and don't want the soon to be trashed segments be sticky in I believe that we don't use `readOnce` for merging, but for tiny metadata files that we load fully in memory. That said, I don't think that this should block this change, we can later look into the proper way to detect whether to pass `MADV_SEQUENTIAL` based on the `IOContext`. -- 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
