Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/1482#issuecomment-54875511
@aarondav Do you have an example scenario where this patch prevents OOM
reporting? Really, the only change here is wrapping the exception logging /
reporting with `if (!Utils.inShutdown())`, which should always return `false`
until we have a fatal error that triggers the shutdown. The actual handling
the fatal error itself should be the same, as far as I can tell.
---
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.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
