rmuir commented on PR #13164: URL: https://github.com/apache/lucene/pull/13164#issuecomment-1982310705
@uschindler was confused how the crashing tests still work. I debugged just now and ensured they are actually `halt()`ing the JVM. The only reason it works is because the crashing tests spawn new java process (which we give them permission to do) with only minimal `-D` args that don't include enabling security manager in the child process. So it is able to be halted and even crashed with Unsafe and setAccessible, but I still think we are doing the right thing to move to `halt()` and remove Unsafe completely from our codebase? -- 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
