Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/6750#discussion_r33827287
--- Diff: core/src/main/scala/org/apache/spark/SparkException.scala ---
@@ -30,3 +30,13 @@ class SparkException(message: String, cause: Throwable)
*/
private[spark] class SparkDriverExecutionException(cause: Throwable)
extends SparkException("Execution error", cause)
+
+/**
+ * Exception indicating an error internal to Spark -- it is in an
inconsistent state, not due
+ * to any error by the user
+ */
+class SparkIllegalStateException(message: String, cause: Throwable)
--- End diff --
I was hoping to create an exception which would make it clear that the user
is totally free from blame. I find IllegalStateExceptions can be confusing --
did this condition result because the user violated some precondition, misused
the api etc? Or is it a spark bug?
I felt this would be more clear, but its not critical.
---
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]
