Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/622#issuecomment-42352943
Hey Mark, one comment on this (also discussed over email): instead of
setting the uncaught exception handler of the Akka thread we happen to run in,
it might be better to wrap these in a try/catch, and the easiest way to do it
would be to add a `Utils.tryOrExit(block: => Unit)` that goes through the same
code paths as here on exceptions. What do you think? It will be more verbose
but it will limit the chance of this uncaught exception handler breaking other
stuff (though in normal operation we shouldn't be randomly getting exceptions
AFAIK).
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
