Skip to content

Commit c8d4762

Browse files
authored
build: Stop running Comet's Spark 4 tests on Linux for PR builds (#1802)
1 parent 3bd1902 commit c8d4762

File tree

2 files changed

+72
-27
lines changed

2 files changed

+72
-27
lines changed

.github/workflows/pr_build.yml

Lines changed: 0 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -149,33 +149,6 @@ jobs:
149149
# upload test reports only for java 17
150150
upload-test-reports: ${{ matrix.java_version == '17' }}
151151

152-
linux-test-with-spark4_0:
153-
strategy:
154-
matrix:
155-
os: [ubuntu-latest]
156-
java_version: [17]
157-
test-target: [java]
158-
spark-version: ['4.0']
159-
is_push_event:
160-
- ${{ github.event_name == 'push' }}
161-
fail-fast: false
162-
name: ${{ matrix.os }}/java ${{ matrix.java_version }}-spark-${{matrix.spark-version}}/${{ matrix.test-target }}
163-
runs-on: ${{ matrix.os }}
164-
container:
165-
image: amd64/rust
166-
steps:
167-
- uses: actions/checkout@v4
168-
- name: Setup Rust & Java toolchain
169-
uses: ./.github/actions/setup-builder
170-
with:
171-
rust-version: ${{env.RUST_VERSION}}
172-
jdk-version: ${{ matrix.java_version }}
173-
- name: Java test steps
174-
uses: ./.github/actions/java-test
175-
with:
176-
maven_opts: -Pspark-${{ matrix.spark-version }}
177-
upload-test-reports: true
178-
179152
linux-test-with-old-spark:
180153
strategy:
181154
matrix:
Lines changed: 72 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,72 @@
1+
# Licensed to the Apache Software Foundation (ASF) under one
2+
# or more contributor license agreements. See the NOTICE file
3+
# distributed with this work for additional information
4+
# regarding copyright ownership. The ASF licenses this file
5+
# to you under the Apache License, Version 2.0 (the
6+
# "License"); you may not use this file except in compliance
7+
# with the License. You may obtain a copy of the License at
8+
#
9+
# http://www.apache.org/licenses/LICENSE-2.0
10+
#
11+
# Unless required by applicable law or agreed to in writing,
12+
# software distributed under the License is distributed on an
13+
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
14+
# KIND, either express or implied. See the License for the
15+
# specific language governing permissions and limitations
16+
# under the License.
17+
18+
name: PR Build (Linux + Spark 4)
19+
20+
concurrency:
21+
group: ${{ github.repository }}-${{ github.head_ref || github.sha }}-${{ github.workflow }}
22+
cancel-in-progress: true
23+
24+
on:
25+
26+
# temporarily disabled on PRs due to https://github.com/apache/datafusion-comet/issues/1786
27+
# push:
28+
# paths-ignore:
29+
# - "doc/**"
30+
# - "docs/**"
31+
# - "**.md"
32+
# pull_request:
33+
# paths-ignore:
34+
# - "doc/**"
35+
# - "docs/**"
36+
# - "**.md"
37+
# manual trigger
38+
# https://docs.github.com/en/actions/managing-workflow-runs/manually-running-a-workflow
39+
workflow_dispatch:
40+
41+
env:
42+
RUST_VERSION: stable
43+
44+
jobs:
45+
46+
linux-test-with-spark4_0:
47+
strategy:
48+
matrix:
49+
os: [ubuntu-latest]
50+
java_version: [17]
51+
test-target: [java]
52+
spark-version: ['4.0']
53+
is_push_event:
54+
- ${{ github.event_name == 'push' }}
55+
fail-fast: false
56+
name: ${{ matrix.os }}/java ${{ matrix.java_version }}-spark-${{matrix.spark-version}}/${{ matrix.test-target }}
57+
runs-on: ${{ matrix.os }}
58+
container:
59+
image: amd64/rust
60+
steps:
61+
- uses: actions/checkout@v4
62+
- name: Setup Rust & Java toolchain
63+
uses: ./.github/actions/setup-builder
64+
with:
65+
rust-version: ${{env.RUST_VERSION}}
66+
jdk-version: ${{ matrix.java_version }}
67+
- name: Java test steps
68+
uses: ./.github/actions/java-test
69+
with:
70+
maven_opts: -Pspark-${{ matrix.spark-version }}
71+
upload-test-reports: true
72+

0 commit comments

Comments
 (0)