This is an automated email from the ASF dual-hosted git repository.
alamb pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/datafusion.git
The following commit(s) were added to refs/heads/main by this push:
new 264f4c51fc Move hash collision test to run only when merging to main.
(#13973)
264f4c51fc is described below
commit 264f4c51fc97981435f1a1827de934472d60edf8
Author: Bruce Ritchie <[email protected]>
AuthorDate: Thu Jan 2 13:34:50 2025 -0500
Move hash collision test to run only when merging to main. (#13973)
---
.github/workflows/hash_collisions.yml | 54 +++++++++++++++++++++++++++++++++++
.github/workflows/rust.yml | 21 --------------
2 files changed, 54 insertions(+), 21 deletions(-)
diff --git a/.github/workflows/hash_collisions.yml
b/.github/workflows/hash_collisions.yml
new file mode 100644
index 0000000000..908bec9d13
--- /dev/null
+++ b/.github/workflows/hash_collisions.yml
@@ -0,0 +1,54 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership. The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License. You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied. See the License for the
+# specific language governing permissions and limitations
+# under the License.
+
+name: Rust Hash Collisions
+
+concurrency:
+ group: ${{ github.repository }}-${{ github.head_ref || github.sha }}-${{
github.workflow }}
+ cancel-in-progress: true
+
+#
https://docs.github.com/en/actions/writing-workflows/choosing-when-your-workflow-runs/events-that-trigger-workflows#running-your-pull_request-workflow-when-a-pull-request-merges
+#
+# this job is intended to only run on merge to main branch
+on:
+ pull_request:
+ branches:
+ - main
+ types:
+ - closed
+
+jobs:
+ # Check answers are correct when hash values collide
+ hash-collisions:
+ name: cargo test hash collisions (amd64)
+ runs-on: ubuntu-latest
+ container:
+ image: amd64/rust
+ if: github.event.pull_request.merged == true
+ steps:
+ - uses: actions/checkout@v4
+ with:
+ submodules: true
+ fetch-depth: 1
+ - name: Setup Rust toolchain
+ uses: ./.github/actions/setup-builder
+ with:
+ rust-version: stable
+ - name: Run tests
+ run: |
+ cd datafusion
+ cargo test --profile ci --exclude datafusion-examples --exclude
datafusion-benchmarks --exclude datafusion-sqllogictest --workspace --lib
--tests --features=force_hash_collisions,avro
diff --git a/.github/workflows/rust.yml b/.github/workflows/rust.yml
index f5b6eece9d..7ac0dfa782 100644
--- a/.github/workflows/rust.yml
+++ b/.github/workflows/rust.yml
@@ -541,27 +541,6 @@ jobs:
- name: Run clippy
run: ci/scripts/rust_clippy.sh
- # Check answers are correct when hash values collide
- hash-collisions:
- name: cargo test hash collisions (amd64)
- needs: linux-build-lib
- runs-on: ubuntu-latest
- container:
- image: amd64/rust
- steps:
- - uses: actions/checkout@v4
- with:
- submodules: true
- fetch-depth: 1
- - name: Setup Rust toolchain
- uses: ./.github/actions/setup-builder
- with:
- rust-version: stable
- - name: Run tests
- run: |
- cd datafusion
- cargo test --profile ci --exclude datafusion-examples --exclude
datafusion-benchmarks --exclude datafusion-sqllogictest --workspace --lib
--tests --features=force_hash_collisions,avro
-
cargo-toml-formatting-checks:
name: check Cargo.toml formatting
needs: linux-build-lib
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
