uschindler commented on PR #11917:
URL: https://github.com/apache/lucene/pull/11917#issuecomment-1314119766

   Maybe we should start again here:
   - sorry for mentioning mlock2, I will think of a way in the future to hook 
into the MemorySegment and configure it in native ways (e.g. set madvise) on 
MMapDirectory. I won't implement a methodhandle to mlock2 without haveing the 
fun of fighting with @rmuir ;-]
   - Maybe split the PR into two parts: (1) adding the improved preload 
BiPredicate (this would allow Elasticsearch to do whatever they want) and (2) 
add the IOContext support for those files and also add the example as proposed 
here.
   
   I would not change the default. That's up to the end-user of Lucene (e.g. 
Elasticsearch, Solr, Opensearch).


-- 
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

Reply via email to