You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@lucene.apache.org by GitBox <gi...@apache.org> on 2022/11/15 10:18:07 UTC

[GitHub] [lucene] uschindler commented on pull request #11930: Introduce IOContext.LOAD

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

   As we started over here: My suggestion would be to let this go in without changing the default in MMapDirectory.
   
   If we want to change the default we can make a followup PR on MMapDirectory. But at moment due to disagreement if preload is enough, we should just add the option to enable this in MMapDir. If we really want to stick the small files and load them to heap, maybe a NRTCachingDirectory like approach would be best.


-- 
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-unsubscribe@lucene.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org