Skip to content

Commit

Permalink
apacheGH-45270: [C++][CI] Disable mimalloc on Valgrind builds (apache…
Browse files Browse the repository at this point in the history
…#45271)

### Rationale for this change

Valgrind is not aware of third-party allocators such as mimalloc and jemalloc. This can lead to spurious errors or, on the contrary, it could hide some actual issues.

### What changes are included in this PR?

Disable mimalloc in the Valgrind build (jemalloc is already disabled).

### Are these changes tested?

Yes, by existing CI.

### Are there any user-facing changes?

No.

* GitHub Issue: apache#45270

Authored-by: Antoine Pitrou <[email protected]>
Signed-off-by: Jacob Wujciak-Jens <[email protected]>
  • Loading branch information
pitrou authored Jan 15, 2025
1 parent 3222e2a commit e434536
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -332,6 +332,7 @@ services:
ARROW_FLIGHT_SQL: "OFF"
ARROW_GANDIVA: "OFF"
ARROW_JEMALLOC: "OFF"
ARROW_MIMALLOC: "OFF"
ARROW_RUNTIME_SIMD_LEVEL: "AVX2" # AVX512 not supported by Valgrind (ARROW-9851)
ARROW_TEST_MEMCHECK: "ON"
ARROW_USE_LD_GOLD: "ON"
Expand Down

0 comments on commit e434536

Please sign in to comment.