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/arrow-rs.git
The following commit(s) were added to refs/heads/main by this push:
new ea8070690e ci: consolidate parquet_derive & parquet-geospatial checks
into parquet workflow (#11167)
ea8070690e is described below
commit ea8070690e5afc9ffd2a1b0eb3865f2acb7b2f67
Author: Jeffrey Vo <[email protected]>
AuthorDate: Fri Sep 25 20:14:29 2026 +0900
ci: consolidate parquet_derive & parquet-geospatial checks into parquet
workflow (#11167)
- similar to https://github.com/apache/arrow-rs/pull/11153 but for
parquet_derive
also figured might as well do the same for parquet-geospatial too
---
.github/workflows/parquet-geospatial.yml | 83 --------------------------------
.github/workflows/parquet.yml | 16 +++++-
.github/workflows/parquet_derive.yml | 70 ---------------------------
3 files changed, 15 insertions(+), 154 deletions(-)
diff --git a/.github/workflows/parquet-geospatial.yml
b/.github/workflows/parquet-geospatial.yml
deleted file mode 100644
index e20ed05884..0000000000
--- a/.github/workflows/parquet-geospatial.yml
+++ /dev/null
@@ -1,83 +0,0 @@
-# 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.
-
----
-# tests for parquet-geospatial crate
-name: "parquet-geospatial"
-
-concurrency:
- group: ${{ github.repository }}-${{ github.head_ref || github.sha }}-${{
github.workflow }}
- cancel-in-progress: true
-
-# trigger for all PRs that touch certain files and changes to main
-on:
- push:
- branches:
- - main
- pull_request:
- paths:
- - parquet-geospatial/**
- - rust-toolchain.toml
- - .github/**
-
-permissions:
- contents: read
-
-jobs:
- # test the crate
- linux-test:
- name: Test
- runs-on: ubuntu-latest
- container:
- image: amd64/rust
- steps:
- - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 #
v7.0.1
- with:
- submodules: true
- - name: Setup Rust toolchain
- uses: ./.github/actions/setup-builder
- - name: Test parquet-geospatial
- run: cargo test -p parquet-geospatial
-
- # test compilation
- linux-features:
- name: Check Compilation
- runs-on: ubuntu-latest
- container:
- image: amd64/rust
- steps:
- - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 #
v7.0.1
- with:
- submodules: true
- - name: Setup Rust toolchain
- uses: ./.github/actions/setup-builder
- - name: Check compilation (parquet-geospatial)
- run: cargo check -p parquet-geospatial
-
- clippy:
- name: Clippy
- runs-on: ubuntu-latest
- container:
- image: amd64/rust
- steps:
- - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 #
v7.0.1
- - name: Setup Rust toolchain
- uses: ./.github/actions/setup-builder
- - name: Setup Clippy
- run: rustup component add clippy
- - name: Run clippy (parquet-geospatial)
- run: cargo clippy -p parquet-geospatial --all-targets --all-features
-- -D warnings
diff --git a/.github/workflows/parquet.yml b/.github/workflows/parquet.yml
index bf6d043bde..c1c8c6704c 100644
--- a/.github/workflows/parquet.yml
+++ b/.github/workflows/parquet.yml
@@ -16,7 +16,7 @@
# under the License.
---
-# tests for parquet and parquet-variant crates
+# tests for parquet and related crates
name: "parquet"
concurrency:
@@ -52,6 +52,8 @@ on:
- arrow-select/**
- arrow-string/**
- parquet/**
+ - parquet_derive/**
+ - parquet_derive_test/**
- parquet-geospatial/**
- parquet-variant/**
- parquet-variant-compute/**
@@ -89,6 +91,12 @@ jobs:
# Test parquet examples
cargo run -p parquet --example read_parquet
cargo run -p parquet --example read_with_rowgroup --features="async"
+ - name: Test parquet_derive
+ run: |
+ cargo test -p parquet_derive
+ cargo test -p parquet_derive_test
+ - name: Test parquet-geospatial
+ run: cargo test -p parquet-geospatial
# test compilation
linux-features:
@@ -149,6 +157,8 @@ jobs:
run: cargo check -p parquet-variant-json
- name: Check compilation (parquet-variant-compute)
run: cargo check -p parquet-variant-compute
+ - name: Check compilation (parquet-geospatial)
+ run: cargo check -p parquet-geospatial
# test the parquet crate builds against wasm32 in stable rust
@@ -222,9 +232,13 @@ jobs:
run: rustup component add clippy
- name: Run clippy
run: cargo clippy -p parquet --all-targets --all-features -- -D
warnings
+ - name: Run clippy (parquet_derive)
+ run: cargo clippy -p parquet_derive --all-features -- -D warnings
- name: Run clippy (parquet-variant)
run: cargo clippy -p parquet-variant --all-targets --all-features --
-D warnings
- name: Run clippy (parquet-variant-json)
run: cargo clippy -p parquet-variant-json --all-targets --all-features
-- -D warnings
- name: Run clippy (parquet-variant-compute)
run: cargo clippy -p parquet-variant-compute --all-targets
--all-features -- -D warnings
+ - name: Run clippy (parquet-geospatial)
+ run: cargo clippy -p parquet-geospatial --all-targets --all-features
-- -D warnings
diff --git a/.github/workflows/parquet_derive.yml
b/.github/workflows/parquet_derive.yml
deleted file mode 100644
index 26665808c1..0000000000
--- a/.github/workflows/parquet_derive.yml
+++ /dev/null
@@ -1,70 +0,0 @@
-# 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.
-
----
-# tests for parquet_derive crate
-name: parquet_derive
-
-concurrency:
- group: ${{ github.repository }}-${{ github.head_ref || github.sha }}-${{
github.workflow }}
- cancel-in-progress: true
-
-# trigger for all PRs that touch certain files and changes to main
-on:
- push:
- branches:
- - main
- pull_request:
- paths:
- - parquet/**
- - parquet_derive/**
- - parquet_derive_test/**
- - rust-toolchain.toml
- - .github/**
-
-permissions:
- contents: read
-
-jobs:
- # test the crate
- linux-test:
- name: Test
- runs-on: ubuntu-latest
- container:
- image: amd64/rust
- steps:
- - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 #
v7.0.1
- with:
- submodules: true
- - name: Setup Rust toolchain
- uses: ./.github/actions/setup-builder
- - name: Test
- run: cargo test -p parquet_derive
-
- clippy:
- name: Clippy
- runs-on: ubuntu-latest
- container:
- image: amd64/rust
- steps:
- - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 #
v7.0.1
- - name: Setup Rust toolchain
- uses: ./.github/actions/setup-builder
- - name: Setup Clippy
- run: rustup component add clippy
- - name: Run clippy
- run: cargo clippy -p parquet_derive --all-features -- -D warnings
