Skip to content

Commit

Permalink
ARROW-11896: [Rust] Disable Debug symbols on CI test builds
Browse files Browse the repository at this point in the history
The theory is that the inclusion of debug symbols is increasing the memory requirements of compiling the test binaries which is causing the tests to hit the CI builder's memory limits (and being OOM killed).

Changes:
Since the debug symbols aren't used for tests, run the CI tests without them to save memory

In theory this might also make the builds faster as well

@nevi-me  and I theorize (without proof) that some new release of a dependency pushed the memory usage over the github builder limit which is why we started seeing this all of a sudden on master without any obvious corresponding code change

Closes apache#9653 from alamb/alamb/test_no_debug_symbols

Authored-by: Andrew Lamb <andrew@nerdnetworks.org>
Signed-off-by: Andrew Lamb <andrew@nerdnetworks.org>
  • Loading branch information
alamb authored and GeorgeAp committed Jun 7, 2021
1 parent cca6d56 commit fdda11d
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions .github/workflows/rust.yml
Expand Up @@ -66,6 +66,8 @@ jobs:
run: |
export CARGO_HOME="/github/home/.cargo"
export CARGO_TARGET_DIR="/github/home/target"
# do not produce debug symbols to keep memory usage down
export RUSTFLAGS="-C debuginfo=0"
cd rust
cargo build
Expand Down Expand Up @@ -108,6 +110,8 @@ jobs:
run: |
export CARGO_HOME="/github/home/.cargo"
export CARGO_TARGET_DIR="/github/home/target"
# do not produce debug symbols to keep memory usage down
export RUSTFLAGS="-C debuginfo=0"
cd rust
# run tests on all workspace members with default feature list
cargo test
Expand Down Expand Up @@ -162,6 +166,8 @@ jobs:
run: |
export CARGO_HOME="/github/home/.cargo"
export CARGO_TARGET_DIR="/github/home/target"
# do not produce debug symbols to keep memory usage down
export RUSTFLAGS="-C debuginfo=0"
cd rust/arrow
cargo test --features "simd"
Expand Down Expand Up @@ -388,5 +394,7 @@ jobs:
run: |
export CARGO_HOME="/github/home/.cargo"
export CARGO_TARGET_DIR="/github/home/target"
# do not produce debug symbols to keep memory usage down
export RUSTFLAGS="-C debuginfo=0"
cd rust/arrow
cargo build --target wasm32-unknown-unknown

0 comments on commit fdda11d

Please sign in to comment.