davidm-db commented on code in PR #47537:
URL: https://github.com/apache/spark/pull/47537#discussion_r1697245949
##########
sql/catalyst/src/main/scala/org/apache/spark/sql/errors/SqlScriptingException.scala:
##########
@@ -17,40 +17,77 @@
package org.apache.spark.sql.errors
-import org.apache.spark.SparkException
+import org.apache.spark.{SparkException, SparkThrowableHelper}
+import org.apache.spark.sql.catalyst.trees.Origin
+import
org.apache.spark.sql.errors.SqlScriptingException.errorMessageWithLineNumber
+
+class SqlScriptingException protected (
+ origin: Origin,
+ errorClass: String,
+ cause: Throwable,
+ messageParameters: Map[String, String] = Map.empty)
+ extends SparkException(
+ message = errorMessageWithLineNumber(origin, errorClass,
messageParameters),
+ errorClass = Option(errorClass),
+ cause = cause,
+ messageParameters = messageParameters
+ ) {
+
+}
/**
* Object for grouping error messages thrown during parsing/interpreting phase
* of the SQL Scripting Language interpreter.
*/
-private[sql] object SqlScriptingErrors extends QueryErrorsBase {
+private[sql] object SqlScriptingException {
Review Comment:
I'm thinking maybe we should leave `SqlScriptingErrors` instead of using
`SqlScriptingException` object - if you look at the `SparkException` as an
example, they use object only for some internal stuff and exceptions, parameter
formatting, etc. while various error classes are used to construct concrete
`SparkExceptions`.
In such case, I would move `SqlScriptingException` to something like
`org.apache.spark.sql.exceptions` - I know it doesn't exist at the moment, but
I think we should create the package since I don't see any more meaningful
place to put the new exception type.
--
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
