HyukjinKwon commented on a change in pull request #29958:
URL: https://github.com/apache/spark/pull/29958#discussion_r500720741
##########
File path: .github/workflows/build_and_test.yml
##########
@@ -319,3 +319,29 @@ jobs:
./dev/change-scala-version.sh 2.13
./build/mvn $MAVEN_CLI_OPTS -DskipTests -Pyarn -Pmesos -Pkubernetes
-Phive -Phive-thriftserver -Phadoop-cloud -Djava.version=11 -Pscala-2.13 install
rm -rf ~/.m2/repository/org/apache/spark
+
+ scala-213-sbt:
+ name: Scala 2.13 build with SBT
Review comment:
I am okay with replacing the existing "Scala 2.13 build". Seems that'll
catch the compilation errors more correctly.
> it will take much more longer time doesn't it?
It does takes longer to detect the errors. However, there are many other
combinations to test and currently we can't test all combinations. So I think
the default common-ish combination was picked to use it as a GitHub Actions and
Jenkins PR builder. For example, JDK 11 is not being tested in GitHub Actions
and Jenkins PR builder.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
