Skip to content

Commit f30a72a

Browse files
committed
CI: Fix presto-on-spark native tests for PRs that only modify docs
1 parent 4f8bff8 commit f30a72a

File tree

1 file changed

+16
-0
lines changed

1 file changed

+16
-0
lines changed

.github/workflows/prestocpp-linux-build-and-unit-test.yml

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -328,43 +328,59 @@ jobs:
328328
MAVEN_TEST: "-B -Dair.check.skip-all -Dmaven.javadoc.skip=true -DLogTestDurationListener.enabled=true --fail-at-end"
329329
steps:
330330
- uses: actions/checkout@v4
331+
if: |
332+
github.event_name == 'schedule' || needs.changes.outputs.codechange == 'true'
331333
332334
- name: Fix git permissions
335+
if: |
336+
github.event_name == 'schedule' || needs.changes.outputs.codechange == 'true'
333337
# Usually actions/checkout does this but as we run in a container
334338
# it doesn't work
335339
run: git config --global --add safe.directory ${GITHUB_WORKSPACE}
336340

337341
- name: Download artifacts
342+
if: |
343+
github.event_name == 'schedule' || needs.changes.outputs.codechange == 'true'
338344
uses: actions/download-artifact@v4
339345
with:
340346
name: presto-native-build
341347
path: presto-native-execution/_build/release
342348

343349
# Permissions are lost when uploading. Details here: https://github.com/actions/upload-artifact/issues/38
344350
- name: Restore execute permissions and library path
351+
if: |
352+
github.event_name == 'schedule' || needs.changes.outputs.codechange == 'true'
345353
run: |
346354
chmod +x ${GITHUB_WORKSPACE}/presto-native-execution/_build/release/presto_cpp/main/presto_server
347355
chmod +x ${GITHUB_WORKSPACE}/presto-native-execution/_build/release/velox/velox/functions/remote/server/velox_functions_remote_server_main
348356
# Ensure transitive dependency libboost-iostreams is found.
349357
ldconfig /usr/local/lib
350358
351359
- name: Install OpenJDK17
360+
if: |
361+
github.event_name == 'schedule' || needs.changes.outputs.codechange == 'true'
352362
uses: actions/setup-java@v4
353363
with:
354364
distribution: 'temurin'
355365
java-version: 17.0.15
356366
cache: 'maven'
357367
- name: Download nodejs to maven cache
368+
if: |
369+
github.event_name == 'schedule' || needs.changes.outputs.codechange == 'true'
358370
run: .github/bin/download_nodejs
359371

360372
- name: Maven install
373+
if: |
374+
github.event_name == 'schedule' || needs.changes.outputs.codechange == 'true'
361375
env:
362376
# Use different Maven options to install.
363377
MAVEN_OPTS: "-Xmx2G -XX:+ExitOnOutOfMemoryError"
364378
run: |
365379
for i in $(seq 1 3); do ./mvnw clean install $MAVEN_FAST_INSTALL -pl 'presto-native-tests' -am && s=0 && break || s=$? && sleep 10; done; (exit $s)
366380
367381
- name: Run presto-on-spark native tests
382+
if: |
383+
github.event_name == 'schedule' || needs.changes.outputs.codechange == 'true'
368384
run: |
369385
export PRESTO_SERVER_PATH="${GITHUB_WORKSPACE}/presto-native-execution/_build/release/presto_cpp/main/presto_server"
370386
export TESTFILES=`find ./presto-native-execution/src/test -type f -name 'TestPrestoSpark*.java'`

0 commit comments

Comments
 (0)