Skip to content

Commit 25ee207

Browse files
committed
Starting testing for java Actions
1 parent c695ef6 commit 25ee207

File tree

1 file changed

+43
-31
lines changed

1 file changed

+43
-31
lines changed

.github/workflows/presto-test.yml

Lines changed: 43 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -2,23 +2,28 @@ name: Presto Test
22

33
on:
44
workflow_dispatch:
5-
inputs:
5+
inputs: &workflow_inputs
66
velox_commit:
77
description: 'Velox commit SHA or branch'
8-
required: true
8+
type: string
9+
required: false
910
default: 'main'
1011
presto_commit:
1112
description: 'Presto commit SHA or branch'
12-
required: true
13-
default: 'main'
14-
workflow_call:
15-
inputs:
16-
velox_commit:
17-
type: string
18-
required: true
19-
presto_commit:
2013
type: string
21-
required: true
14+
required: false
15+
default: 'master'
16+
presto_worker_type:
17+
description: 'Type of presto worker to use'
18+
type: choice
19+
required: false
20+
options:
21+
- java
22+
- native-cpu
23+
- native-gpu
24+
default: 'java'
25+
workflow_call:
26+
*workflow_inputs
2227

2328
jobs:
2429
build-and-test:
@@ -31,32 +36,39 @@ jobs:
3136
uses: actions/checkout@v4
3237
with:
3338
repository: facebookincubator/velox
34-
ref: ${{ inputs.velox_commit || github.event.inputs.velox_commit }}
35-
36-
- name: Build Velox with cuDF
37-
run: |
38-
git config --global --add safe.directory .
39-
# this is wrong, needs to be the same as in velox-test.yml
40-
# both need to use the same script
41-
mkdir -p velox-build && cd velox-build
42-
cmake -G Ninja -DVELOX_ENABLE_CUDF=ON ..
43-
ninja
39+
ref: ${{ inputs.velox_commit }}
40+
path: velox
4441

4542
- name: Checkout Presto
4643
uses: actions/checkout@v4
4744
with:
4845
repository: prestodb/presto
49-
ref: ${{ inputs.presto_commit || github.event.inputs.presto_commit }}
46+
ref: ${{ inputs.presto_commit }}
5047
path: presto
5148

52-
- name: Build Presto with Velox Connector
49+
- name: Build and run java presto
50+
if: ${{ github.event.inputs.presto_worker_type == 'java' }}
5351
run: |
54-
# TODO: Untested, replace with working code
55-
cd presto
56-
git config --global --add safe.directory .
57-
./mvnw clean package -Pvelox -DskipTests
58-
59-
- name: Run Presto Integration Tests
52+
cd presto/scripts
53+
./start_java_presto.sh
54+
sleep 30
55+
./run_integ_test.sh
56+
./stop_presto.sh
57+
58+
- name: Build and run native-cpu presto
59+
if: ${{ github.event.inputs.presto_worker_type == 'native-cpu' }}
60+
run: |
61+
cd presto/scripts
62+
./start_native_cpu_presto.sh
63+
sleep 30
64+
./run_integ_test.sh
65+
./stop_presto.sh
66+
67+
- name: Build and run native-gpu presto
68+
if: ${{ github.event.inputs.presto_worker_type == 'native-gpu' }}
6069
run: |
61-
cd presto
62-
./presto-server/bin/run-tests.sh
70+
cd presto/scripts
71+
./start_native_gpu_presto.sh
72+
sleep 30
73+
./run_integ_test.sh
74+
./stop_presto.sh

0 commit comments

Comments
 (0)