Skip to content

Commit 9757455

Browse files
committed
CI on forked branches
1 parent 13230be commit 9757455

2 files changed

Lines changed: 46 additions & 9 deletions

File tree

.github/workflows/build_and_test.yml

Lines changed: 44 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -114,9 +114,8 @@ jobs:
114114
container:
115115
image: amd64/rust
116116
steps:
117-
- uses: actions/checkout@v6
118-
- name: Setup Rust & Java toolchain
119-
uses: ./.github/actions/setup-builder
117+
- name: Checkout and setup builder
118+
uses: ./.github/actions/checkout-and-setup-builder
120119
with:
121120
rust-version: ${{ env.RUST_VERSION }}
122121
jdk-version: 17
@@ -149,3 +148,45 @@ jobs:
149148
with:
150149
name: test-results-sql_core-1-spark${{ inputs.spark-short }}-jdk${{ inputs.java }}
151150
path: "**/target/test-reports/*.xml"
151+
152+
sql_core-2:
153+
needs: precondition
154+
name: "Build modules: sql_core-2"
155+
runs-on: ubuntu-24.04
156+
container:
157+
image: amd64/rust
158+
steps:
159+
- name: Checkout and setup builder
160+
uses: ./.github/actions/checkout-and-setup-builder
161+
with:
162+
rust-version: ${{ env.RUST_VERSION }}
163+
jdk-version: 17
164+
- name: Download native library
165+
uses: actions/download-artifact@v8
166+
with:
167+
name: native-lib-linux
168+
path: native/target/release/
169+
- name: Setup Spark
170+
uses: ./.github/actions/setup-spark-builder
171+
with:
172+
spark-version: ${{ inputs.spark-full }}
173+
spark-short-version: ${{ inputs.spark-short }}
174+
skip-native-build: true
175+
- name: Run Spark sql_core-2 tests
176+
run: |
177+
cd apache-spark
178+
rm -rf /root/.m2/repository/org/apache/parquet
179+
export SERIAL_SBT_TESTS=1
180+
NOLINT_ON_COMPILE=true ENABLE_COMET=true ENABLE_COMET_ONHEAP=true \
181+
build/sbt -Dsbt.log.noformat=true -mem $SBT_MEM \
182+
'set Global / concurrentRestrictions := Seq(Tags.limit(Tags.ForkedTestGroup, 1))' \
183+
"sql/testOnly * -- -n org.apache.spark.tags.ExtendedSQLTest"
184+
env:
185+
LC_ALL: "C.UTF-8"
186+
SBT_MEM: "3072"
187+
- name: Upload test results to report
188+
if: always()
189+
uses: actions/upload-artifact@v7
190+
with:
191+
name: test-results-sql_core-2-spark${{ inputs.spark-short }}-jdk${{ inputs.java }}
192+
path: "**/target/test-reports/*.xml"

.github/workflows/update_build_status.yml

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -15,11 +15,7 @@
1515
# KIND, either express or implied. See the License for the
1616
# specific language governing permissions and limitations
1717
# under the License.
18-
19-
# Polls each open PR's fork-side "Build" workflow run and PATCHes the
20-
# corresponding apache/datafusion-comet check run so the PR check status
21-
# stays in sync with the fork-hosted CI. Modelled on Apache Spark's
22-
# update_build_status.yml.
18+
#
2319

2420
name: Update build status workflow
2521

@@ -51,7 +47,7 @@ jobs:
5147
const maybeReady = ['behind', 'clean', 'draft', 'has_hooks', 'unknown', 'unstable'];
5248
5349
// Iterate open PRs
54-
for await (const prs of github.paginate.iterator(endpoint, params)) {
50+
for await (const prs of github.paginate.iterator(endpoint,params)) {
5551
// Each page
5652
for await (const pr of prs.data) {
5753
console.log('SHA: ' + pr.head.sha)

0 commit comments

Comments
 (0)