vibhatha commented on a change in pull request #12672:
URL: https://github.com/apache/arrow/pull/12672#discussion_r836434947
##########
File path: python/CMakeLists.txt
##########
@@ -69,6 +69,7 @@ endif()
if("${CMAKE_SOURCE_DIR}" STREQUAL "${CMAKE_CURRENT_SOURCE_DIR}")
option(PYARROW_BUILD_CUDA "Build the PyArrow CUDA support" OFF)
option(PYARROW_BUILD_FLIGHT "Build the PyArrow Flight integration" OFF)
+ option(PYARROW_BUILD_ENGINE "Build the PyArrow Substrait integration" OFF)
Review comment:
@lidavidm Good question. Here I followed the format followed in
integrating Substrait work in Cmakes. In the CMakes, we see `ARROW_ENGINE` to
check if `engine` is enabled. I assume here there could be many engines and
substrait could be one of them. Is there a better standard we can follow here?
cc @westonpace
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
