LuciferYang commented on code in PR #37345:
URL: https://github.com/apache/spark/pull/37345#discussion_r933829350
##########
core/src/main/scala/org/apache/spark/deploy/SparkHadoopUtil.scala:
##########
@@ -23,7 +23,6 @@ import java.text.DateFormat
import java.util.{Arrays, Date, Locale}
import scala.collection.JavaConverters._
-import scala.collection.immutable.Map
Review Comment:
I check Scala 2.13 as follows:
```
dev/change-scala-version.sh 2.13
mvn clean install -pl core -Pscala-2.13 -am
-Dtest.exclude.tags=org.apache.spark.tags.ExtendedLevelDBTest
```
```
[INFO]
------------------------------------------------------------------------
[INFO] Reactor Summary for Spark Project Parent POM 3.4.0-SNAPSHOT:
[INFO]
[INFO] Spark Project Parent POM ........................... SUCCESS [ 4.233
s]
[INFO] Spark Project Tags ................................. SUCCESS [ 8.936
s]
[INFO] Spark Project Local DB ............................. SUCCESS [ 12.578
s]
[INFO] Spark Project Networking ........................... SUCCESS [ 53.431
s]
[INFO] Spark Project Shuffle Streaming Service ............ SUCCESS [ 12.492
s]
[INFO] Spark Project Unsafe ............................... SUCCESS [ 10.007
s]
[INFO] Spark Project Launcher ............................. SUCCESS [ 10.775
s]
[INFO] Spark Project Core ................................. SUCCESS [33:39
min]
[INFO]
------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO]
------------------------------------------------------------------------
[INFO] Total time: 35:32 min
[INFO] Finished at: 2022-07-30T22:29:15+08:00
[INFO]
------------------------------------------------------------------------
```
Build and test passed and I also check in branch 2.13 that Predef still
exists
https://github.com/scala/scala/blob/cbc012e573346dc685c478eec5fcbb56d22ea884/src/library/scala/Predef.scala#L157-L164
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
