Skip to content

Commit

Permalink
Enable arrow filesystem when building tests.
Browse files Browse the repository at this point in the history
  • Loading branch information
vyasr committed Dec 16, 2022
1 parent d285dd6 commit e830685
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
1 change: 1 addition & 0 deletions cpp/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -161,6 +161,7 @@ include(cmake/thirdparty/get_thrust.cmake)
# find rmm
include(cmake/thirdparty/get_rmm.cmake)
# find arrow
set(CUDF_ENABLE_ARROW_FILESYSTEM ${BUILD_TESTS})
include(cmake/thirdparty/get_arrow.cmake)
# find dlpack
include(cmake/thirdparty/get_dlpack.cmake)
Expand Down
5 changes: 3 additions & 2 deletions cpp/cmake/thirdparty/get_arrow.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ endfunction()

# This function finds arrow and sets any additional necessary environment variables.
function(find_and_configure_arrow VERSION BUILD_STATIC ENABLE_S3 ENABLE_ORC ENABLE_PYTHON
ENABLE_PARQUET
ENABLE_PARQUET ENABLE_FILESYSTEM
)

if(USE_LIBARROW_FROM_PYARROW)
Expand Down Expand Up @@ -166,6 +166,7 @@ function(find_and_configure_arrow VERSION BUILD_STATIC ENABLE_S3 ENABLE_ORC ENAB
# e.g. needed by blazingsql-io
${ARROW_PARQUET_OPTIONS}
"ARROW_PARQUET ${ENABLE_PARQUET}"
"ARROW_FILESYSTEM ${CUDF_ENABLE_ARROW_FILESYSTEM}"
${ARROW_PYTHON_OPTIONS}
# Arrow modifies CMake's GLOBAL RULE_LAUNCH_COMPILE unless this is off
"ARROW_USE_CCACHE OFF"
Expand Down Expand Up @@ -372,5 +373,5 @@ endif()

find_and_configure_arrow(
${CUDF_VERSION_Arrow} ${CUDF_USE_ARROW_STATIC} ${CUDF_ENABLE_ARROW_S3} ${CUDF_ENABLE_ARROW_ORC}
${CUDF_ENABLE_ARROW_PYTHON} ${CUDF_ENABLE_ARROW_PARQUET}
${CUDF_ENABLE_ARROW_PYTHON} ${CUDF_ENABLE_ARROW_PARQUET} ${CUDF_ENABLE_ARROW_FILESYSTEM}
)

0 comments on commit e830685

Please sign in to comment.