Skip to content

Commit 0b557b3

Browse files
sarutakHyukjinKwon
authored andcommitted
[SPARK-33265][TEST] Rename classOf[Seq] to classOf[scala.collection.Seq] in PostgresIntegrationSuite for Scala 2.13
### What changes were proposed in this pull request? This PR renames some part of `Seq` in `PostgresIntegrationSuite` to `scala.collection.Seq`. When I run `docker-integration-test`, I noticed that `PostgresIntegrationSuite` failed due to `ClassCastException`. The reason is the same as what is resolved in SPARK-29292. ### Why are the changes needed? To pass `docker-integration-test` for Scala 2.13. ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Ran `PostgresIntegrationSuite` fixed and confirmed it successfully finished. Closes apache#30166 from sarutak/fix-toseq-postgresql. Authored-by: Kousuke Saruta <[email protected]> Signed-off-by: HyukjinKwon <[email protected]>
1 parent ff724d2 commit 0b557b3

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/PostgresIntegrationSuite.scala

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -116,14 +116,14 @@ class PostgresIntegrationSuite extends DockerJDBCIntegrationSuite {
116116
assert(classOf[java.lang.Boolean].isAssignableFrom(types(7)))
117117
assert(classOf[String].isAssignableFrom(types(8)))
118118
assert(classOf[String].isAssignableFrom(types(9)))
119-
assert(classOf[Seq[Int]].isAssignableFrom(types(10)))
120-
assert(classOf[Seq[String]].isAssignableFrom(types(11)))
121-
assert(classOf[Seq[Double]].isAssignableFrom(types(12)))
122-
assert(classOf[Seq[BigDecimal]].isAssignableFrom(types(13)))
119+
assert(classOf[scala.collection.Seq[Int]].isAssignableFrom(types(10)))
120+
assert(classOf[scala.collection.Seq[String]].isAssignableFrom(types(11)))
121+
assert(classOf[scala.collection.Seq[Double]].isAssignableFrom(types(12)))
122+
assert(classOf[scala.collection.Seq[BigDecimal]].isAssignableFrom(types(13)))
123123
assert(classOf[String].isAssignableFrom(types(14)))
124124
assert(classOf[java.lang.Float].isAssignableFrom(types(15)))
125125
assert(classOf[java.lang.Short].isAssignableFrom(types(16)))
126-
assert(classOf[Seq[BigDecimal]].isAssignableFrom(types(17)))
126+
assert(classOf[scala.collection.Seq[BigDecimal]].isAssignableFrom(types(17)))
127127
assert(rows(0).getString(0).equals("hello"))
128128
assert(rows(0).getInt(1) == 42)
129129
assert(rows(0).getDouble(2) == 1.25)

0 commit comments

Comments
 (0)