kou commented on PR #46509:
URL: https://github.com/apache/arrow/pull/46509#issuecomment-2918477941

   > let sanitizer instrument third-party libraries 
   
   This means that we add `-fsanitize=thread` to bundled dependencies, right?
   
   Does 
https://github.com/google/sanitizers/wiki/threadsanitizercppmanual#non-instrumented-code
 require for system dependencies (such as system opentelemetry-cpp, not bundled 
opentelemetry-cpp) too? If so, we need to choose "the suppression way". Because 
we can't control system dependencies.
   
   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`.)


-- 
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

Reply via email to