Skip to content

Commit

Permalink
More cleanups.
Browse files Browse the repository at this point in the history
  • Loading branch information
jorgecarleitao committed Apr 18, 2021
1 parent eed1b82 commit 72b2f9b
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 10 deletions.
13 changes: 6 additions & 7 deletions .github/workflows/rust.yml
Expand Up @@ -70,14 +70,13 @@ jobs:
run: |
export CARGO_HOME="/github/home/.cargo"
export CARGO_TARGET_DIR="/github/home/target"
cd rust
cargo build
# Ballista is currently not part of the main workspace so requires a separate build step
- name: Build Ballista
run: |
export CARGO_HOME="/github/home/.cargo"
export CARGO_TARGET_DIR="/github/home/target"
cd rust/ballista/rust
cd ballista/rust
# snmalloc requires cmake so build without default features
cargo build --no-default-features
Expand All @@ -96,8 +95,8 @@ jobs:
# 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"
ARROW_TEST_DATA: /__w/arrow/arrow/testing/data
PARQUET_TEST_DATA: /__w/arrow/arrow/cpp/submodules/parquet-testing/data
ARROW_TEST_DATA: /__w/arrow-rs/arrow-rs/testing/data
PARQUET_TEST_DATA: /__w/arrow-rs/arrow-rs/parquet-testing/data
steps:
- uses: actions/checkout@v2
with:
Expand Down Expand Up @@ -135,7 +134,7 @@ jobs:
run: |
export CARGO_HOME="/github/home/.cargo"
export CARGO_TARGET_DIR="/github/home/target"
cd rust/ballista/rust
cd ballista/rust
# snmalloc requires cmake so build without default features
cargo test --no-default-features
Expand All @@ -161,7 +160,7 @@ jobs:
shell: bash
run: |
export ARROW_TEST_DATA=$(pwd)/testing/data
export PARQUET_TEST_DATA=$(pwd)/cpp/submodules/parquet-testing/data
export PARQUET_TEST_DATA=$(pwd)/parquet-testing/data
# do not produce debug symbols to keep memory usage down
export RUSTFLAGS="-C debuginfo=0"
cargo test
Expand Down Expand Up @@ -285,7 +284,7 @@ jobs:
export CARGO_TARGET_DIR="/home/runner/target"
export ARROW_TEST_DATA=$(pwd)/testing/data
export PARQUET_TEST_DATA=$(pwd)/cpp/submodules/parquet-testing/data
export PARQUET_TEST_DATA=$(pwd)/parquet-testing/data
# 2020-11-15: There is a cargo-tarpaulin regression in 0.17.0
# see https://github.com/xd009642/tarpaulin/issues/618
Expand Down
6 changes: 3 additions & 3 deletions ballista/rust/benchmarks/tpch/README.md
Expand Up @@ -41,7 +41,7 @@ To run the benchmarks it is necessary to have at least one Ballista scheduler an
To run the scheduler from source:

```bash
cd $ARROW_HOME/rust/ballista/rust/scheduler
cd $ARROW_HOME/ballista/rust/scheduler
RUST_LOG=info cargo run --release
```

Expand All @@ -50,7 +50,7 @@ By default the scheduler will bind to `0.0.0.0` and listen on port 50050.
To run the executor from source:

```bash
cd $ARROW_HOME/rust/ballista/rust/executor
cd $ARROW_HOME/ballista/rust/executor
RUST_LOG=info cargo run --release
```

Expand All @@ -65,7 +65,7 @@ RUST_LOG=info RUSTFLAGS='-C target-cpu=native -C lto -C codegen-units=1 -C embed
To run the benchmarks:

```bash
cd $ARROW_HOME/rust/ballista/rust/benchmarks/tpch
cd $ARROW_HOME/ballista/rust/benchmarks/tpch
cargo run --release benchmark --host localhost --port 50050 --query 1 --path $(pwd)/data --format tbl
```

Expand Down

0 comments on commit 72b2f9b

Please sign in to comment.