Skip to content

Commit

Permalink
ARROW-9789: [C++] Don't install jemalloc in parallel
Browse files Browse the repository at this point in the history
On ARROW-6437 (apache#7928) we saw occasional "File exists" errors on `jemalloc_ep` on macOS. Googling the error message led back to ARROW-739 (apache#456), which fixed this before by forcing install with `-j1`. ARROW-3492 later made it so jemalloc would build (but not install) in parallel. Then ARROW-3539 (apache#2779) was addressing a problem with that change and, along with fixing the build parallelization issue, unfortunately reverted the original `make -j1 install` fix.

This patch restores the fix from ARROW-739.

Closes apache#7995 from nealrichardson/jemalloc-install

Authored-by: Neal Richardson <neal.p.richardson@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
  • Loading branch information
nealrichardson authored and emkornfield committed Oct 16, 2020
1 parent 357dda5 commit 8331130
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cpp/cmake_modules/ThirdpartyToolchain.cmake
Expand Up @@ -1454,7 +1454,7 @@ if(ARROW_JEMALLOC)
BUILD_IN_SOURCE 1
BUILD_COMMAND ${JEMALLOC_BUILD_COMMAND}
BUILD_BYPRODUCTS "${JEMALLOC_STATIC_LIB}"
INSTALL_COMMAND ${MAKE} install)
INSTALL_COMMAND ${MAKE} -j1 install)

# Don't use the include directory directly so that we can point to a path
# that is unique to our codebase.
Expand Down

0 comments on commit 8331130

Please sign in to comment.