This is an automated email from the ASF dual-hosted git repository.
dongjoon 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 39fc6108bfa [SPARK-45562][DOCS] Regenerate
`docs/sql-error-conditions.md` and add `42KDF` to `SQLSTATE table` in
`error/README.md`
39fc6108bfa is described below
commit 39fc6108bfaaa0ce471f6460880109f948ba5c62
Author: yangjie01 <[email protected]>
AuthorDate: Wed Nov 15 09:04:25 2023 -0800
[SPARK-45562][DOCS] Regenerate `docs/sql-error-conditions.md` and add
`42KDF` to `SQLSTATE table` in `error/README.md`
### What changes were proposed in this pull request?
This pr re-generate `docs/sql-error-conditions.md` and add `42KDF` to
`SQLSTATE table` in `error/README.md` refer to
https://docs.gcp.databricks.com/error-messages/sqlstates.html
<img width="855" alt="image"
src="https://github.com/apache/spark/assets/1475305/6e302ea5-fa61-40a3-99ef-7c6d63334194";>
### Why are the changes needed?
Make GA pass
https://github.com/apache/spark/actions/runs/6874376893/job/18695941494
```
[info] - SQLSTATE invariants *** FAILED *** (30 milliseconds)
[info] fx.apply(s) was false 42KDF (SparkThrowableSuite.scala:74)
[info] org.scalatest.exceptions.TestFailedException:
[info] at
org.scalatest.Assertions.newAssertionFailedException(Assertions.scala:472)
[info] at
org.scalatest.Assertions.newAssertionFailedException$(Assertions.scala:471)
[info] at
org.scalatest.Assertions$.newAssertionFailedException(Assertions.scala:1231)
[info] at
org.scalatest.Assertions$AssertionsHelper.macroAssert(Assertions.scala:1295)
[info] at
org.apache.spark.SparkThrowableSuite.$anonfun$checkCondition$1(SparkThrowableSuite.scala:74)
[info] at scala.collection.immutable.List.foreach(List.scala:333)
[info] at
org.apache.spark.SparkThrowableSuite.checkCondition(SparkThrowableSuite.scala:73)
[info] at
org.apache.spark.SparkThrowableSuite.$anonfun$new$6(SparkThrowableSuite.scala:138)
[info] at
scala.runtime.java8.JFunction0$mcV$sp.apply(JFunction0$mcV$sp.scala:18)
[info] at
org.scalatest.enablers.Timed$$anon$1.timeoutAfter(Timed.scala:127)
....
[info] - Error classes match with document *** FAILED *** (71 milliseconds)
[info] "...gs-error-class.html)[
[info]
[info] ### XML_ROW_TAG_MISSING
[info]
[info] [SQLSTATE:
42KDF](sql-error-conditions-sqlstates.html#class-42-syntax-error-or-access-rule-violation)
[info]
[info] `<rowTag>` option is required for reading files in XML format.]"
did not equal "...gs-error-class.html)[]" The error class document is not up to
date. Please regenerate it. (SparkThrowableSuite.scala:346)
[info] Analysis:
[info] "...gs-error-class.html)[
[info]
[info] ### XML_ROW_TAG_MISSING
[info]
[info] [SQLSTATE:
42KDF](sql-error-conditions-sqlstates.html#class-42-syntax-error-or-access-rule-violation)
[info]
[info] `<rowTag>` option is required for reading files in XML format.]" ->
"...gs-error-class.html)[]"
```
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
- Pass GitHub Actions
- Manual check `build/sbt "core/testOnly *SparkThrowableSuite"`
**Before**
```
[info] Run completed in 1 second, 205 milliseconds.
[info] Total number of tests run: 19
[info] Suites: completed 1, aborted 0
[info] Tests: succeeded 17, failed 2, canceled 0, ignored 0, pending 0
[info] *** 2 TESTS FAILED ***
[error] Failed tests:
[error] org.apache.spark.SparkThrowableSuite
```
**After**
```
[info] Run completed in 1 second, 131 milliseconds.
[info] Total number of tests run: 19
[info] Suites: completed 1, aborted 0
[info] Tests: succeeded 19, failed 0, canceled 0, ignored 0, pending 0
[info] All tests passed.
```
### Was this patch authored or co-authored using generative AI tooling?
No
Closes #43817 from LuciferYang/SPARK-45562-FOLLOWUP.
Authored-by: yangjie01 <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
---
common/utils/src/main/resources/error/README.md | 1 +
docs/sql-error-conditions.md | 6 ++++++
2 files changed, 7 insertions(+)
diff --git a/common/utils/src/main/resources/error/README.md
b/common/utils/src/main/resources/error/README.md
index 8d8529bea56..7b1b9038aeb 100644
--- a/common/utils/src/main/resources/error/README.md
+++ b/common/utils/src/main/resources/error/README.md
@@ -895,6 +895,7 @@ The following SQLSTATEs are collated from:
|42KDC |42 |Syntax error or Access Rule violation |KDC
|Archived file reference. |Databricks |N
|Databricks
|
|42KDD |42 |Syntax error or Access Rule violation |KDD
|Unsupported operation in streaming view. |Databricks |N
|Databricks
|
|42KDE |42 |Syntax error or Access Rule violation |KDE
|Unsupported operation on streaming dataset. |Databricks |N
|Databricks
|
+|42KDF |42 |Syntax error or Access Rule violation |KDF |A
required routine parameter is missing an argument. |Databricks |N
|Databricks
|
|42P01 |42 |Syntax error or Access Rule violation |P01
|undefined_table |PostgreSQL |N
|PostgreSQL Redshift
|
|42P02 |42 |Syntax Error or Access Rule Violation |P02
|undefined_parameter |PostgreSQL |N
|PostgreSQL Redshift
|
|42P03 |42 |Syntax Error or Access Rule Violation |P03
|duplicate_cursor |PostgreSQL |N
|PostgreSQL Redshift
|
diff --git a/docs/sql-error-conditions.md b/docs/sql-error-conditions.md
index f23b4fb78b6..cba6a24b869 100644
--- a/docs/sql-error-conditions.md
+++ b/docs/sql-error-conditions.md
@@ -2375,3 +2375,9 @@ The operation `<operation>` requires a `<requiredType>`.
But `<objectName>` is a
The `<functionName>` requires `<expectedNum>` parameters but the actual number
is `<actualNum>`.
For more details see
[WRONG_NUM_ARGS](sql-error-conditions-wrong-num-args-error-class.html)
+
+### XML_ROW_TAG_MISSING
+
+[SQLSTATE:
42KDF](sql-error-conditions-sqlstates.html#class-42-syntax-error-or-access-rule-violation)
+
+`<rowTag>` option is required for reading files in XML format.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
