rmuir opened a new pull request, #13277:
URL: https://github.com/apache/lucene/pull/13277

   Background:
   
   Historically IndexWriter treated OutOfMemoryError special, for defensive 
reasons. It was expanded to VirtualMachineError, to try to play it safe in 
similar disastrous circumstances.
   
   We should treat any Error as a tragedy, as it isn't an Exception, and it 
isn't something a "reasonable" application should catch. IndexWriter should be 
reasonable. See #7049 for some of the reasoning.
   
   We can't pretend this will detect any possible scenario that might cause 
harm, e.g. a jvm bug might simply miscompile some code and cause silent 
corruption. But we should try harder by playing by the rules.
   
   Closes #13275
   


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