jpountz commented on PR #14156: URL: https://github.com/apache/lucene/pull/14156#issuecomment-2607399239
> Seems we just trade an isLoaded for an madvise on systems with enough memory? This is correct. I made this suggestion because it was similar to your initial proposal: skipping the `MS::isLoaded` check and doing a `madvise` call directly instead. > Maybe the trick here would be to keep track of the loaded state in a way that works across slices so that we throttle more efficiently? I'm good with exploring sharing state across slices as well. I optimized for simplicity with the current implementation, but we may be able to figure out something that works well across slices as well. -- 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
