MaxGekk opened a new pull request, #44464:
URL: https://github.com/apache/spark/pull/44464

   ### What changes were proposed in this pull request?
   In the PR, I propose to create `SparkThrowable` sub-classes only with an 
error class by making the constructor with `message` private.
   
   ### Why are the changes needed?
   To improve user experience with Spark SQL by unifying error exceptions: the 
final goal is all Spark exception should contain an error class.
   
   ### Does this PR introduce _any_ user-facing change?
   No since user's code shouldn't throw `SparkThrowable` sub-classes but it can 
if it depends on error message formats.
   
   ### How was this patch tested?
   By existing test test suites like:
   ```
   $ PYSPARK_PYTHON=python3 build/sbt "sql/testOnly 
org.apache.spark.sql.SQLQueryTestSuite"
   ```
   
   ### Was this patch authored or co-authored using generative AI tooling?
   No.


-- 
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

Reply via email to