This is an automated email from the ASF dual-hosted git repository.
maxgekk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 50955cb934ff [SPARK-48358][SQL][TESTS][FOLLOWUP] Use `condition`
instead of `errorClass` in `SqlScriptingInterpreterSuite`
50955cb934ff is described below
commit 50955cb934ff3ac8d970f15b6e3e69f598235b08
Author: Max Gekk <[email protected]>
AuthorDate: Wed Sep 11 11:27:01 2024 +0200
[SPARK-48358][SQL][TESTS][FOLLOWUP] Use `condition` instead of `errorClass`
in `SqlScriptingInterpreterSuite`
### What changes were proposed in this pull request?
In the PR, I propose to replace `errorClass` by `condition` in
`SqlScriptingInterpreterSuite`
### Why are the changes needed?
The changes from the PR https://github.com/apache/spark/pull/47756 conflict
to https://github.com/apache/spark/pull/48027
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
By running the modified test:
```
$ build/sbt "test:testOnly *SqlScriptingInterpreterSuite"
```
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #48072 from MaxGekk/fix-errorClass-REPEAT.
Authored-by: Max Gekk <[email protected]>
Signed-off-by: Max Gekk <[email protected]>
---
.../apache/spark/sql/scripting/SqlScriptingInterpreterSuite.scala | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git
a/sql/core/src/test/scala/org/apache/spark/sql/scripting/SqlScriptingInterpreterSuite.scala
b/sql/core/src/test/scala/org/apache/spark/sql/scripting/SqlScriptingInterpreterSuite.scala
index b703e77d4d73..8d9cd1d8c780 100644
---
a/sql/core/src/test/scala/org/apache/spark/sql/scripting/SqlScriptingInterpreterSuite.scala
+++
b/sql/core/src/test/scala/org/apache/spark/sql/scripting/SqlScriptingInterpreterSuite.scala
@@ -670,7 +670,7 @@ class SqlScriptingInterpreterSuite extends QueryTest with
SharedSparkSession {
exception = intercept[SqlScriptingException] (
runSqlScript(commands)
),
- errorClass = "INVALID_BOOLEAN_STATEMENT",
+ condition = "INVALID_BOOLEAN_STATEMENT",
parameters = Map("invalidStatement" -> "1")
)
}
@@ -693,7 +693,7 @@ class SqlScriptingInterpreterSuite extends QueryTest with
SharedSparkSession {
exception = intercept[SqlScriptingException] (
runSqlScript(commands)
),
- errorClass = "BOOLEAN_STATEMENT_WITH_EMPTY_ROW",
+ condition = "BOOLEAN_STATEMENT_WITH_EMPTY_ROW",
parameters = Map("invalidStatement" -> "(SELECT * FROM T)")
)
}
@@ -719,7 +719,7 @@ class SqlScriptingInterpreterSuite extends QueryTest with
SharedSparkSession {
exception = intercept[SparkException] (
runSqlScript(commands)
),
- errorClass = "SCALAR_SUBQUERY_TOO_MANY_ROWS",
+ condition = "SCALAR_SUBQUERY_TOO_MANY_ROWS",
parameters = Map.empty,
context = ExpectedContext(fragment = "(SELECT * FROM t)", start = 141,
stop = 157)
)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
