rmuir commented on PR #13146: URL: https://github.com/apache/lucene/pull/13146#issuecomment-1977219947
We should ban it in forbidden-apis if not already... also some of the usages in tests should be reviewed. Two of them are the JVM-crashing tests. Maybe it is enough to just `Runtime.halt()` these days? Some of this was paranoia around things such as shutdown-hooks closing open file handles and the like, I think it might be irrelevant? The other is in RAMUsageTester, I don't understand it enough. -- 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
