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 c47a9205b2d [SPARK-45477][INFRA] Use `matrix.java/inputs.java` to
replace the hardcoded Java version in `test results/unit tests log` naming
c47a9205b2d is described below
commit c47a9205b2de1c0638e3824564f0d09bad3f4b24
Author: yangjie01 <yangji...@baidu.com>
AuthorDate: Tue Oct 10 08:57:18 2023 -0700
[SPARK-45477][INFRA] Use `matrix.java/inputs.java` to replace the hardcoded
Java version in `test results/unit tests log` naming
### What changes were proposed in this pull request?
There are some places in `build_and_test.yml` where the Java version part
is hardcoded to 8 when naming `test results/unit tests log`, this pr changes
them to use `matrix.java/inputs.java`.
### Why are the changes needed?
The actual Java version should be used to name the `test results/unit tests
log`.
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
Monitor GA, check if the `test results/unit tests log` has used the actual
Java version for naming.
### Was this patch authored or co-authored using generative AI tooling?
No
Closes #43305 from LuciferYang/ga-test-log-name.
Authored-by: yangjie01 <yangji...@baidu.com>
Signed-off-by: Dongjoon Hyun <dh...@apple.com>
---
.github/workflows/build_and_test.yml | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/.github/workflows/build_and_test.yml
b/.github/workflows/build_and_test.yml
index b9bce98b4d2..52c0b12ce0a 100644
--- a/.github/workflows/build_and_test.yml
+++ b/.github/workflows/build_and_test.yml
@@ -465,13 +465,13 @@ jobs:
if: always()
uses: actions/upload-artifact@v3
with:
- name: test-results-${{ matrix.modules }}--8-${{ inputs.hadoop
}}-hive2.3
+ name: test-results-${{ matrix.modules }}--${{ matrix.java }}-${{
inputs.hadoop }}-hive2.3
path: "**/target/test-reports/*.xml"
- name: Upload unit tests log files
if: ${{ !success() }}
uses: actions/upload-artifact@v3
with:
- name: unit-tests-log-${{ matrix.modules }}--8-${{ inputs.hadoop
}}-hive2.3
+ name: unit-tests-log-${{ matrix.modules }}--${{ matrix.java }}-${{
inputs.hadoop }}-hive2.3
path: "**/target/unit-tests.log"
sparkr:
@@ -550,7 +550,7 @@ jobs:
if: always()
uses: actions/upload-artifact@v3
with:
- name: test-results-sparkr--8-${{ inputs.hadoop }}-hive2.3
+ name: test-results-sparkr--${{ inputs.java }}-${{ inputs.hadoop
}}-hive2.3
path: "**/target/test-reports/*.xml"
breaking-changes-buf:
@@ -916,13 +916,13 @@ jobs:
if: always()
uses: actions/upload-artifact@v3
with:
- name: test-results-tpcds--8-${{ inputs.hadoop }}-hive2.3
+ name: test-results-tpcds--${{ inputs.java }}-${{ inputs.hadoop
}}-hive2.3
path: "**/target/test-reports/*.xml"
- name: Upload unit tests log files
if: ${{ !success() }}
uses: actions/upload-artifact@v3
with:
- name: unit-tests-log-tpcds--8-${{ inputs.hadoop }}-hive2.3
+ name: unit-tests-log-tpcds--${{ inputs.java }}-${{ inputs.hadoop
}}-hive2.3
path: "**/target/unit-tests.log"
docker-integration-tests:
@@ -983,13 +983,13 @@ jobs:
if: always()
uses: actions/upload-artifact@v3
with:
- name: test-results-docker-integration--8-${{ inputs.hadoop }}-hive2.3
+ name: test-results-docker-integration--${{ inputs.java }}-${{
inputs.hadoop }}-hive2.3
path: "**/target/test-reports/*.xml"
- name: Upload unit tests log files
if: ${{ !success() }}
uses: actions/upload-artifact@v3
with:
- name: unit-tests-log-docker-integration--8-${{ inputs.hadoop }}-hive2.3
+ name: unit-tests-log-docker-integration--${{ inputs.java }}-${{
inputs.hadoop }}-hive2.3
path: "**/target/unit-tests.log"
k8s-integration-tests:
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org
