This is an automated email from the ASF dual-hosted git repository.
dongjoon pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new e9e7eee [SPARK-37120][BUILD][FOLLOWUP] Use configured Java version in
SQL/Hive module runs
e9e7eee is described below
commit e9e7eeee6866235aabff75b227499b003325c068
Author: Dongjoon Hyun <[email protected]>
AuthorDate: Mon Nov 8 13:52:17 2021 -0800
[SPARK-37120][BUILD][FOLLOWUP] Use configured Java version in SQL/Hive
module runs
### What changes were proposed in this pull request?
This is a follow-up of https://github.com/apache/spark/pull/34508 .
This PR aims to use the configured Java version in SQL/Hive module runs too.
### Why are the changes needed?
Java 11/17 is used correctly in the scheduled jobs except `included
modules` like SQL/Hive.
- https://github.com/apache/spark/runs/4139231636?check_suite_focus=true
(Java 11)
- https://github.com/apache/spark/runs/4141378544?check_suite_focus=true
(Java 17)
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
N/A
Closes #34525 from dongjoon-hyun/SPARK-37120-2.
Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
---
.github/workflows/build_and_test.yml | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/.github/workflows/build_and_test.yml
b/.github/workflows/build_and_test.yml
index f0709c9..41ffd23 100644
--- a/.github/workflows/build_and_test.yml
+++ b/.github/workflows/build_and_test.yml
@@ -137,26 +137,26 @@ jobs:
include:
# Hive tests
- modules: hive
- java: 8
+ java: ${{ needs.configure-jobs.outputs.java }}
hadoop: ${{ needs.configure-jobs.outputs.hadoop }}
hive: hive2.3
included-tags: org.apache.spark.tags.SlowHiveTest
comment: "- slow tests"
- modules: hive
- java: 8
+ java: ${{ needs.configure-jobs.outputs.java }}
hadoop: ${{ needs.configure-jobs.outputs.hadoop }}
hive: hive2.3
excluded-tags: org.apache.spark.tags.SlowHiveTest
comment: "- other tests"
# SQL tests
- modules: sql
- java: 8
+ java: ${{ needs.configure-jobs.outputs.java }}
hadoop: ${{ needs.configure-jobs.outputs.hadoop }}
hive: hive2.3
included-tags: org.apache.spark.tags.ExtendedSQLTest
comment: "- slow tests"
- modules: sql
- java: 8
+ java: ${{ needs.configure-jobs.outputs.java }}
hadoop: ${{ needs.configure-jobs.outputs.hadoop }}
hive: hive2.3
excluded-tags: org.apache.spark.tags.ExtendedSQLTest
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
