juliuszsompolski commented on code in PR #42338:
URL: https://github.com/apache/spark/pull/42338#discussion_r1284499359
##########
python/pyspark/errors/exceptions/connect.py:
##########
@@ -188,3 +188,10 @@ class SparkUpgradeException(SparkConnectGrpcException,
BaseSparkUpgradeException
"""
Exception thrown because of Spark upgrade from Spark Connect.
"""
+
+
+class SparkConnectRetryException(SparkConnectException):
+ """
+ An exception that can be thrown upstream when inside retry and which will
be retryable
+ regardless of policy.
+ """
Review Comment:
nit: if this is an internal exception that is being thrown only inside
Retrying, and not an user facing exception, should this be placed somewhere
next to the AttemptManager and Retrying impl?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
