spark icon indicating copy to clipboard operation
spark copied to clipboard

[SPARK-49953][CORE] Require `errorClass` in `SparkException`

Open MaxGekk opened this issue 4 months ago • 2 comments

What changes were proposed in this pull request?

In the PR, I propose to remove the constructors of SparkException that accept a plan string as an error message, and leave only constructors with the error classes.

Why are the changes needed?

To migrate the code which uses SparkException on new error framework.

Does this PR introduce any user-facing change?

No, it shouldn't because the exception is supposed to raised by Spark.

How was this patch tested?

By running the modified test suites:

$ build/sbt "sql/testOnly org.apache.spark.sql.SQLQueryTestSuite"

Was this patch authored or co-authored using generative AI tooling?

No.

MaxGekk avatar Oct 13 '24 12:10 MaxGekk