You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by ks...@apache.org on 2019/01/25 10:38:07 UTC
[arrow] branch master updated: ARROW-4329: Python should include
the parquet headers
This is an automated email from the ASF dual-hosted git repository.
kszucs pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git
The following commit(s) were added to refs/heads/master by this push:
new 32f91b2 ARROW-4329: Python should include the parquet headers
32f91b2 is described below
commit 32f91b2c923809f8413c3a28efe0d0e0545fcc21
Author: Peter <cH...@netflix.com>
AuthorDate: Fri Jan 25 11:37:39 2019 +0100
ARROW-4329: Python should include the parquet headers
When using PYARROW_BUILD_PARQUET the parquet headers should be included along
side the arrow and plasma headers.
Author: Peter <cH...@netflix.com>
Closes #3450 from cHYzZQo/python-parquet-headers and squashes the following commits:
ce744c5d <Peter> ARROW-4329: Python should include the parquet and gandiva headers
---
python/CMakeLists.txt | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/python/CMakeLists.txt b/python/CMakeLists.txt
index 3a048cc..7d2882e 100644
--- a/python/CMakeLists.txt
+++ b/python/CMakeLists.txt
@@ -414,6 +414,10 @@ if (PYARROW_BUILD_PARQUET)
endif()
include_directories(SYSTEM ${PARQUET_INCLUDE_DIR})
+ if (PYARROW_BUNDLE_ARROW_CPP)
+ file(COPY ${PARQUET_INCLUDE_DIR}/parquet DESTINATION ${BUILD_OUTPUT_ROOT_DIRECTORY}/include)
+ endif()
+
if (PYARROW_PARQUET_USE_SHARED)
if (PYARROW_BUNDLE_ARROW_CPP)
bundle_arrow_lib(PARQUET_SHARED_LIB
@@ -504,6 +508,8 @@ if (PYARROW_BUILD_GANDIVA)
include_directories(SYSTEM ${GANDIVA_INCLUDE_DIR})
if (PYARROW_BUNDLE_ARROW_CPP)
+ file(COPY ${GANDIVA_INCLUDE_DIR}/gandiva DESTINATION ${BUILD_OUTPUT_ROOT_DIRECTORY}/include)
+
bundle_arrow_lib(GANDIVA_SHARED_LIB
ABI_VERSION ${ARROW_ABI_VERSION}
SO_VERSION ${ARROW_SO_VERSION})