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)

Reply via email to