itholic commented on code in PR #39591:
URL: https://github.com/apache/spark/pull/39591#discussion_r1071744763
##########
python/pyspark/sql/utils.py:
##########
@@ -18,17 +18,24 @@
import os
from typing import Any, Callable, Optional, Sequence, TYPE_CHECKING, cast,
TypeVar
-import py4j
from py4j.java_collections import JavaArray
from py4j.java_gateway import (
JavaClass,
JavaGateway,
JavaObject,
- is_instance_of,
)
-from py4j.protocol import Py4JJavaError
from pyspark import SparkContext
+from pyspark.errors import ( # noqa: F401
Review Comment:
Sure, let me add a comment
##########
python/docs/source/reference/pyspark.errors.rst:
##########
@@ -27,3 +27,14 @@ Errors
PySparkException.getErrorClass
PySparkException.getMessageParameters
+ CapturedException.getErrorClass
+ CapturedException.getMessageParameters
+ CapturedException.getSqlState
Review Comment:
`PySparkException` has no `getSqlState` method, because SQLSTATE is only
exists in errors from JVM side, which are captured by `CapturedException`
Think maybe do you mean
```
PySparkException.getErrorClass
PySparkException.getMessageParameters
CapturedException.getSqlState
```
?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
