C0urante commented on PR #12666: URL: https://github.com/apache/kafka/pull/12666#issuecomment-1297554373
@ijuma Sorry for the delay. The tradeoff involved is whether we want to account for direct calls to `System::exit` or not. If we're confident enough in our review process to catch those before they get merged, or at least to quickly identify them after they've been merged and started causing issues in CI, then it should be fine to remove the security manager and move everything into the `Exit` class. I'll give that a shot and see if I can push a new draft by the end of the week. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
