mridulm commented on a change in pull request #30391:
URL: https://github.com/apache/spark/pull/30391#discussion_r525893835
##########
File path: .github/workflows/build_and_test.yml
##########
@@ -433,11 +407,32 @@ jobs:
key: scala-213-coursier-${{ hashFiles('**/pom.xml', '**/plugins.sbt')
}}
restore-keys: |
scala-213-coursier-
- - name: Install Java 11
+ - name: Install Java 8
uses: actions/setup-java@v1
with:
- java-version: 11
+ java-version: 8
- name: Build with SBT
run: |
./dev/change-scala-version.sh 2.13
./build/sbt -Pyarn -Pmesos -Pkubernetes -Phive -Phive-thriftserver
-Phadoop-cloud -Pkinesis-asl -Djava.version=11 -Pscala-2.13 compile test:compile
+
Review comment:
@HyukjinKwon Currently, we merge if either of the builds pass (jenkins
or github).
The reason I was requesting @dongjoon-hyun to move it to top was if jenkins
gives positive build (since it is not testing hadoop 2), then only way to
identify hadoop 2.x failure is by github actions failing on hadoop 2 build.
If we can get jenkins to also build hadoop-2, then we dont need to move this
to top.
Else, it is common for reviewers to not see the last entry in github build
(when there is a jenkins green).
----------------------------------------------------------------
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]
