GH-39973: [C++][CI] Disable debug memory pool for ASAN and Valgrind#39975
GH-39973: [C++][CI] Disable debug memory pool for ASAN and Valgrind#39975pitrou merged 10 commits intoapache:mainfrom
Conversation
|
cc @pitrou |
|
If things go well, we should be seeing the ASAN failure as described in #39976 somewhere in CI job. This is expected as we are strengthening the ASAN check so hidden legacy failures are exposed. |
|
@github-actions crossbow submit -g cpp |
|
I've cleaned up the failed Crossbow requests :-) |
This comment was marked as outdated.
This comment was marked as outdated.
|
The legacy ASAN failure will be gone once #39994 goes in. |
This reverts commit 43cf728a2f72ad034ff93031e9e06c79d7d9eca9.
This reverts commit 1302f4f52fc3fe576f39a1bd349817b9db2cf24b.
|
I've rebased, hopefully CI wil be green now. |
|
@github-actions crossbow submit -g cpp |
|
Revision: b9ff2f3 Submitted crossbow builds: ursacomputing/crossbow @ actions-1f71ea13ea |
Thank you for rebasing. Praying for CI now. |
|
You can stop praying now :-) |
|
After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit a946214. There were 4 benchmark results indicating a performance regression:
The full Conbench report has more details. It also includes information about 1 possible false positive for unstable benchmarks that are known to sometimes produce them. |
…rind (apache#39975) ### Rationale for this change Disable debug memory pool for ASAN and Valgrind so that they can detect more subtle memory issues regarding to buffer tail bytes. ### What changes are included in this PR? 1. Add a `none` option to debug memory pool env var to make other things slightly easier. 2. Change `*_test.sh` scripts to conditionally set debug memory pool env var. 3. Top-level docker compose change to pass none to debug memory pool env var for ASAN and Valgrind. ### Are these changes tested? The CI should cover it well. ### Are there any user-facing changes? No. * Closes: apache#39973 Authored-by: Ruoxi Sun <zanmato1984@gmail.com> Signed-off-by: Antoine Pitrou <antoine@python.org>
Rationale for this change
Disable debug memory pool for ASAN and Valgrind so that they can detect more subtle memory issues regarding to buffer tail bytes.
What changes are included in this PR?
noneoption to debug memory pool env var to make other things slightly easier.*_test.shscripts to conditionally set debug memory pool env var.Are these changes tested?
The CI should cover it well.
Are there any user-facing changes?
No.