Skip to content

Conversation

@wesm
Copy link
Member

@wesm wesm commented Jun 26, 2017

I will make a corresponding PR to parquet-cpp to ensure that this code migration is complete enough.

Change-Id: Id761068793f6b6e0ece1741349693ecd30ae6b67
Change-Id: Ia313f8673be451982b2e5952f17d2676ecb24dea
Change-Id: I7667374393ba7f064aa125de1d2e4532e37479d0
Copy link
Member

@xhochy xhochy left a comment

Choose a reason for hiding this comment

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

+1. LGTM

@asfgit asfgit closed this in b065228 Jun 27, 2017
@wesm wesm deleted the ARROW-1154 branch June 27, 2017 12:14
asfgit pushed a commit to apache/parquet-cpp that referenced this pull request Jun 27, 2017
…-1154

I verified locally that this works on Linux with the current iteration of apache/arrow#785, will update the commit in ThirdpartyToolchain.cmake after that is merged to get a passing build

Author: Wes McKinney <wes.mckinney@twosigma.com>

Closes #364 from wesm/PARQUET-1045 and squashes the following commits:

fede8d9 [Wes McKinney] Update Arrow version to include ARROW-1159
b44b4ab [Wes McKinney] Fix benchmark. Only build Zlib on Windows when building Thrift
83d2289 [Wes McKinney] Update Arrow version
8914c16 [Wes McKinney] Remove code that was moved to Apache Arrow, refactoring
wesm added a commit to wesm/arrow that referenced this pull request Sep 8, 2018
…-1154

I verified locally that this works on Linux with the current iteration of apache#785, will update the commit in ThirdpartyToolchain.cmake after that is merged to get a passing build

Author: Wes McKinney <wes.mckinney@twosigma.com>

Closes apache#364 from wesm/PARQUET-1045 and squashes the following commits:

fede8d9 [Wes McKinney] Update Arrow version to include ARROW-1159
b44b4ab [Wes McKinney] Fix benchmark. Only build Zlib on Windows when building Thrift
83d2289 [Wes McKinney] Update Arrow version
8914c16 [Wes McKinney] Remove code that was moved to Apache Arrow, refactoring

Change-Id: Ibee94d0dd9d0b6844bdbb3c57da5126c97760108
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