kaivalnp commented on PR #14178:
URL: https://github.com/apache/lucene/pull/14178#issuecomment-2981931299

   Thanks @mikemccand, I've made the suggested changes + rebased + improved 
some documentation!
   
   > One could maybe use ulimit so the kernel will return null if the process 
tries to allocate too much RAM, and subtract the JVM heap from that total.
   
   Interesting, I'll try to do this for HNSW (but RAM usage may be wildly 
different for other types of indexes / transforms / etc.)
   
   > Anyway, I don't think this is a blocker for merging to sandbox -- we can 
learn over time the RAM usage.
   
   +1 -- hopefully we'll discover / estimate its usage better over time..
   
   > I don't think I have karma for it
   
   I see, I've switched back to 
[conda-incubator/setup-miniconda](https://github.com/conda-incubator/setup-miniconda)
 so that we can test the codec regularly..
   It does approximately the same thing as 
[mamba-org/setup-micromamba](https://github.com/mamba-org/setup-micromamba), 
but slower / with more overhead -- however this GH action is already allowed by 
Lucene for some reason :)


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