Skip to content

Conversation

@xhochy
Copy link
Member

@xhochy xhochy commented Mar 5, 2017

Depends on apache/parquet-cpp#265

With this change we can also build self-contained OSX wheels, still we have to find a way to build them reproducibly (will take care of that soon).

xhochy added 3 commits March 5, 2017 19:47
Change-Id: Ic22daecd17771769c27f6feecce83a4b8ebd81fe
Change-Id: I52e713196ef74e3f2cb1b9a0da214622f61e333b
Change-Id: Ia07b4f277ed8a62ed52e49c4066bc73d23b4cd62
OFF)
option(PYARROW_BUNDLE_ARROW_CPP
"Bundle the Arrow C++ libraries"
OFF)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, I'm hoping to soon move the python/src/pyarrow tree to cpp/src/arrow/python, any idea how this will be impacted or whether that will make things any easier?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will have to refactor this a bit but it won't change much sadly. Packaging binary libs in a manylinux1 wheel is cumbersome.

@wesm
Copy link
Member

wesm commented Mar 7, 2017

+1

@asfgit asfgit closed this in b109a24 Mar 7, 2017
wesm pushed a commit to wesm/arrow that referenced this pull request Sep 8, 2018
Closes apache#358. This only includes an Arrow version bump to pick up ARROW-1143

Author: Uwe L. Korn <uwe@apache.org>
Author: Wes McKinney <wes.mckinney@twosigma.com>

Closes apache#360 from wesm/PARQUET-1041 and squashes the following commits:

3f8f0bc [Wes McKinney] Bump Arrow version to master
c134bd6 [Uwe L. Korn] Fix int conversion
1def8a4 [Uwe L. Korn] PARQUET-1041: Support Arrow's NullArray

Change-Id: I4ea8b9055e5b625bc1e65125a0d4e24b1cb06b7a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants