Github user sameeragarwal commented on the issue:
https://github.com/apache/spark/pull/21342
LGTM, Thanks! I've seen variations of this bug in the past as well and am
not aware of a better way to fix this issue.
```scala
scala> Await.result(Future(throw new RuntimeException("foo")), 50.seconds)
java.lang.RuntimeException: foo // regular exceptions fail fast
```
```scala
scala> Await.result(Future(throw new OutOfMemoryError("foo")), 50.seconds)
java.util.concurrent.TimeoutException: Futures timed out after [50 seconds]
// OOME times out
```
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
