[SPARK-32473] Use === instead IndexSeqView

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.1.0
• Fix Version/s: 3.1.0
• Component/s: Spark Core, Tests
• Assignee: Dongjoon Hyun
• Reporter: Dongjoon Hyun
• Created: 29/Jul/20 03:44
• Updated: 29/Jul/20 05:32
• Resolved: 29/Jul/20 05:32

Description

This issue aims to fix `SorterSuite` and `RadixSortSuite` in Scala 2.13 by using `sameElements` instead of `IndexSeqView.==`.

Scala 2.13 reimplements `IndexSeqView` and the behavior is different.

• https://docs.scala-lang.org/overviews/core/collections-migration-213.html

Welcome to Scala 2.12.10 (OpenJDK 64-Bit Server VM, Java 1.8.0_262).
Type in expressions for evaluation. Or try :help.

scala> Seq(1,2,3).toArray.view == Seq(1,2,3).toArray.view
res0: Boolean = true

Welcome to Scala 2.13.3 (OpenJDK 64-Bit Server VM, Java 1.8.0_262).
Type in expressions for evaluation. Or try :help.

scala> Seq(1,2,3).toArray.view == Seq(1,2,3).toArray.view
val res0: Boolean = false

$ dev/change-scala-version.sh 2.13
$ build/mvn test -pl core --am -Pscala-2.13 -Dtest=none -DwildcardSuites=org.apache.spark.util.collection.unsafe.sort.RadixSortSuite
...
Tests: succeeded 9, failed 36, canceled 0, ignored 0, pending 0
*** 36 TESTS FAILED ***

$ build/mvn test -pl core --am -Pscala-2.13 -Dtest=none -DwildcardSuites=org.apache.spark.util.collection.SorterSuite
...
Tests: succeeded 3, failed 1, canceled 0, ignored 2, pending 0
*** 1 TEST FAILED ***

Issue Links

• links to: [Github] Pull Request #29280 (dongjoon-hyun)