Skip to content

Commit 908ae9a

Browse files
authored
chore: Speed up "PR Builds" CI workflows (#1807)
1 parent d598833 commit 908ae9a

File tree

4 files changed

+186
-332
lines changed

4 files changed

+186
-332
lines changed

.github/actions/java-test/action.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,10 @@ inputs:
2626
description: 'Maven options passed to the mvn command'
2727
required: false
2828
default: ''
29+
scan_impl:
30+
description: 'The default Parquet scan implementation'
31+
required: false
32+
default: 'native_comet'
2933
upload-test-reports:
3034
description: 'Whether to upload test results including coverage to GitHub'
3135
required: false
@@ -58,11 +62,15 @@ runs:
5862
- name: Run all tests
5963
shell: bash
6064
if: ${{ inputs.suites == '' }}
65+
env:
66+
COMET_PARQUET_SCAN_IMPL: ${{ inputs.scan_impl }}
6167
run: |
6268
MAVEN_OPTS="-XX:+UnlockDiagnosticVMOptions -XX:+ShowMessageBoxOnError -XX:+HeapDumpOnOutOfMemoryError -XX:ErrorFile=./hs_err_pid%p.log" SPARK_HOME=`pwd` ./mvnw -B clean install ${{ inputs.maven_opts }}
6369
- name: Run specified tests
6470
shell: bash
6571
if: ${{ inputs.suites != '' }}
72+
env:
73+
COMET_PARQUET_SCAN_IMPL: ${{ inputs.scan_impl }}
6674
run: |
6775
MAVEN_SUITES="$(echo "${{ inputs.suites }}" | paste -sd, -)"
6876
echo "Running with MAVEN_SUITES=$MAVEN_SUITES"

.github/workflows/pr_build.yml

Lines changed: 0 additions & 317 deletions
This file was deleted.

0 commit comments

Comments
 (0)