mxm commented on pull request #13042: URL: https://github.com/apache/flink/pull/13042#issuecomment-668083787
Good questions @tillrohrmann. I was looking into setting a default exception handler but then realized we have exception handling in numerous places and it was not exactly trivial to figure out if the default handler would end up being called. Also, you mentioned an application can set a default exception handler too. In the end, the SecurityManager approach seemed to be the most elegant. It has the drawback of needing the permission to install a SecurityManager. Similarly, `Thread#setDefaultUncaughtExceptionHandler` also needs permission from the SecurityManager, if present. There is a bit of magic involved here, but not by default. Overall, I think it is reasonable to allow converting exit calls if explicitly switched on. After all, the configuration option is a workaround for a JVM bug which means that a bit of magic might be appropriate. We can expand the configuration option to be a global option and not only affect the task managers. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
