Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20602#discussion_r167993164
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/StreamExecution.scala
---
@@ -369,7 +370,11 @@ abstract class StreamExecution(
// exception
// UncheckedExecutionException - thrown by codes that cannot throw
a checked
// ExecutionException, such as
BiFunction.apply
- case e2 @ (_: UncheckedIOException | _: ExecutionException | _:
UncheckedExecutionException)
+ // SparkException - thrown if the interrupt happens in the middle
of an RPC wait
+ case e2 @ (_: UncheckedIOException |
+ _: ExecutionException |
+ _: UncheckedExecutionException |
+ _: SparkException)
--- End diff --
I am not sure especially about this. It can be thrown for a great number of
cases which aren't necessarily caused by the stop operation. I don't think it
is a good idea to add it.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
