zanmato1984 commented on PR #46509:
URL: https://github.com/apache/arrow/pull/46509#issuecomment-2918489304
> This means that we add `-fsanitize=thread` to bundled dependencies, right?
Yes.
> Does [Wiki: threadsanitizercppmanual (non instrumented code)
(google/sanitizers)](https://github.com/google/sanitizers/wiki/threadsanitizercppmanual#non-instrumented-code)
require for system dependencies (such as system opentelemetry-cpp, not bundled
opentelemetry-cpp) too?
I never tried but would guess so.
> If so, we need to choose "the suppression way". Because we can't control
system dependencies.
Yes, this is probably safer.
> BTW, we can use `CLANG_TOOLS=18 LLVM=18 UBUNTU=24.04 archery docker run
ubuntu-cpp-thread-sanitizer` on local instead of using CI by `@github-actions
crossbow submit -g cpp`. (We can use `@github-actions crossbow submit
test-ubuntu-24.04-cpp-thread-sanitizer` to run only
`test-ubuntu-24.04-cpp-thread-sanitizer`.)
Very helpful, thank you!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
