rmuir commented on issue #14408: URL: https://github.com/apache/lucene/issues/14408#issuecomment-2755385165
A better default would also be possible, I think, if individual codecs such as vectors, did not set this directly, but instead allowed the Directory class to be in control. User/Directory is only partially in control now, there is a `org.apache.lucene.store.defaultReadAdvice` system property, but then this codec just ignores that, which I don't think is good. So a user has to write custom Directory and mess around with IOContext to correct the issue, which isn't obvious. Let the defaults be as smart as they need. Maybe check `/sys/kernel/mm/lru_gen/enabled` as part of the decision-making! But IMO let the user have the final say, in an easy way. -- 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
