Neal Richardson created ARROW-6977: -------------------------------------- Summary: [C++] Only enable jemalloc background_thread if feature is supported Key: ARROW-6977 URL: https://issues.apache.org/jira/browse/ARROW-6977 Project: Apache Arrow Issue Type: Bug Components: C++ Environment: macOS 10.14, Homebrew Reporter: Neal Richardson Fix For: 1.0.0
Followup to ARROW-6910. When loading the R package after that patch merged, I get this new message: {code} $ R > library(arrow) <jemalloc>: option background_thread currently supports pthread only {code} https://github.com/jemalloc/jemalloc/blob/3d84bd57f4954a17059bd31330ec87d3c1876411/src/background_thread.c#L884-L887 is where the message comes from. Tracing that further, {{have_background_thread}} comes from https://github.com/jemalloc/jemalloc/blob/21cfe59ff7b10a61dabe26cd3dbfb7a255e1f5e8/include/jemalloc/internal/jemalloc_preamble.h.in#L205-L211, which gets set in {{configure.ac}} here: https://github.com/jemalloc/jemalloc/blob/d2dddfb82aac9f2212922eb90324e84790704bfe/configure.ac#L2155-L2157 In sum, on my system, that flag doesn't get set, so {{have_background_thread}} is false, and when that is false and the {{background_thread}} option is true, I get that message printed. And I do not want to see that message. cc [~wesm] -- This message was sent by Atlassian Jira (v8.3.4#803005)