- Notifications
You must be signed in to change notification settings - Fork29.1k
Commit4f9e6ed
committed
### What changes were proposed in this pull request?This PR aims to add `mysql-connector-j` test dependency to `sql/core` module.### Why are the changes needed?`JDBCSuite` requires `mysql-connector-j` client. Previously, it seems that `MariaDB` connector are used unintentionally. We should fix this.https://github.com/apache/spark/blob/cb3ff7234c2c649bc0b6bb4b1c3465fb1e955600/sql/core/src/test/scala/org/apache/spark/sql/jdbc/JDBCSuite.scala#L1413-L1420### Does this PR introduce _any_ user-facing change?No.### How was this patch tested?Pass the CIs.**BEFORE**```$ build/sbt "sql/testOnly org.apache.spark.sql.jdbc.JDBCSuite -- -zSPARK-18419"...[info] JDBCSuite:08:32:54.125 WARN org.apache.hadoop.util.NativeCodeLoader: Unable to load native-hadoop library for your platform... using builtin-java classes where applicable08:32:55.894 ERROR org.apache.spark.sql.execution.datasources.jdbc.connection.ConnectionProvider: Failed to load built-in provider.[info] -SPARK-18419: Fix `asConnectionProperties` to filter case-insensitively *** FAILED *** (13 milliseconds)[info] java.sql.SQLException: No suitable driver...```**AFTER**```$ build/sbt "sql/testOnly org.apache.spark.sql.jdbc.JDBCSuite -- -zSPARK-18419"...[info] JDBCSuite:[info] -SPARK-18419: Fix `asConnectionProperties` to filter case-insensitively (13 milliseconds)[info] Run completed in 2 seconds, 967 milliseconds.[info] Total number of tests run: 1[info] Suites: completed 1, aborted 0[info] Tests: succeeded 1, failed 0, canceled 0, ignored 0, pending 0[info] All tests passed.[success] Total time: 19 s, completed Feb 18, 2026, 8:32:08 AM```### Was this patch authored or co-authored using generative AI tooling?No.Closes#54367 from dongjoon-hyun/SPARK-55590.Authored-by: Dongjoon Hyun <dongjoon@apache.org>Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
1 parentcb3ff72 commit4f9e6ed
1 file changed
+6
-0
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
190 | 190 | | |
191 | 191 | | |
192 | 192 | | |
| 193 | + | |
| 194 | + | |
| 195 | + | |
| 196 | + | |
| 197 | + | |
| 198 | + | |
193 | 199 | | |
194 | 200 | | |
195 | 201 | | |
| |||
0 commit comments
Comments
(0)