From 33c6e6183c1c09594b71db57e10d4eb2e4752544 Mon Sep 17 00:00:00 2001 From: Andy Grove Date: Thu, 12 May 2022 06:59:09 -0600 Subject: [PATCH 1/3] move ballista tests into separate step --- .github/workflows/rust.yml | 38 +++++++++++++++++++++++++++++++++++++- 1 file changed, 37 insertions(+), 1 deletion(-) diff --git a/.github/workflows/rust.yml b/.github/workflows/rust.yml index 056e83df0c407..ab5a24b152a30 100644 --- a/.github/workflows/rust.yml +++ b/.github/workflows/rust.yml @@ -123,9 +123,9 @@ jobs: run: | export ARROW_TEST_DATA=$(pwd)/testing/data export PARQUET_TEST_DATA=$(pwd)/parquet-testing/data + cargo test --no-default-features # test datafusion examples cd datafusion-examples - cargo test --no-default-features cargo run --example csv_sql cargo run --example parquet_sql cargo run --example avro_sql --features=datafusion/avro @@ -134,6 +134,42 @@ jobs: env: CARGO_HOME: "/github/home/.cargo" CARGO_TARGET_DIR: "/github/home/target" + + # test the crate + ballista-test: + name: Test Ballista on AMD64 Rust ${{ matrix.rust }} + needs: [linux-build-lib] + runs-on: ubuntu-latest + strategy: + matrix: + arch: [amd64] + rust: [stable] + container: + image: ${{ matrix.arch }}/rust + env: + # Disable full debug symbol generation to speed up CI build and keep memory down + # "1" means line tables only, which is useful for panic tracebacks. + RUSTFLAGS: "-C debuginfo=1" + steps: + - uses: actions/checkout@v2 + with: + submodules: true + - name: Cache Cargo + uses: actions/cache@v2 + with: + path: /github/home/.cargo + # this key equals the ones on `linux-build-lib` for re-use + key: cargo-cache- + - name: Cache Rust dependencies + uses: actions/cache@v2 + with: + path: /github/home/target + # this key equals the ones on `linux-build-lib` for re-use + key: ${{ runner.os }}-${{ matrix.arch }}-target-cache-${{ matrix.rust }} + - name: Setup Rust toolchain + uses: ./.github/actions/setup-builder + with: + rust-version: ${{ matrix.rust }} # Ballista is currently not part of the main workspace so requires a separate test step - name: Run Ballista tests run: | From 144c738346605778f68953621d9d0e1e37b798eb Mon Sep 17 00:00:00 2001 From: Andy Grove Date: Thu, 12 May 2022 07:27:38 -0600 Subject: [PATCH 2/3] fix --- .github/workflows/rust.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/rust.yml b/.github/workflows/rust.yml index ab5a24b152a30..adbc73d7b0f84 100644 --- a/.github/workflows/rust.yml +++ b/.github/workflows/rust.yml @@ -123,7 +123,7 @@ jobs: run: | export ARROW_TEST_DATA=$(pwd)/testing/data export PARQUET_TEST_DATA=$(pwd)/parquet-testing/data - cargo test --no-default-features + cargo test --no-default-features --features standalone # test datafusion examples cd datafusion-examples cargo run --example csv_sql @@ -135,7 +135,7 @@ jobs: CARGO_HOME: "/github/home/.cargo" CARGO_TARGET_DIR: "/github/home/target" - # test the crate + # run ballista tests ballista-test: name: Test Ballista on AMD64 Rust ${{ matrix.rust }} needs: [linux-build-lib] From cf50d4146c4ddb3b2204639ba5981d8faa4cc160 Mon Sep 17 00:00:00 2001 From: Andy Grove Date: Thu, 12 May 2022 07:33:07 -0600 Subject: [PATCH 3/3] fix --- .github/workflows/rust.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/rust.yml b/.github/workflows/rust.yml index adbc73d7b0f84..f512f26b68ae0 100644 --- a/.github/workflows/rust.yml +++ b/.github/workflows/rust.yml @@ -123,7 +123,7 @@ jobs: run: | export ARROW_TEST_DATA=$(pwd)/testing/data export PARQUET_TEST_DATA=$(pwd)/parquet-testing/data - cargo test --no-default-features --features standalone + cargo test # test datafusion examples cd datafusion-examples cargo run --example csv_sql