Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[C++][Parquet] 11, cpplint cleanup, package target and header installation #42252

Closed
asfimport opened this issue Dec 24, 2015 · 4 comments
Closed

Comments

@asfimport
Copy link
Collaborator

asfimport commented Dec 24, 2015

I'm planning to work on building out parquet-cpp with columnar data structures (see Arrow proposal) for materialized in-memory data and feature complete reader/writers so that native-code consumers like Python can finally read and write Parquet files at native speeds. It would be great to have all this officially a part of Apache Parquet.

This adds minimal support to be able to install the resulting libparquet.so and its various header files to support minimally viable development on downstream C++ and Python projects that will need to depend on this. It also builds in C++11 mode and passes Google's cpplint.

Reporter: Wes McKinney / @wesm
Assignee: Wes McKinney / @wesm

Related issues:

Externally tracked issue: apache/parquet-cpp#14

Note: This issue was originally created as PARQUET-416. Please see the migration documentation for further details.

@asfimport
Copy link
Collaborator Author

Wes McKinney / @wesm:
I'm working on getting Travis CI working again on Linux and OS X for this and then would appreciate if someone would take a look. Will keep you posted

@asfimport
Copy link
Collaborator Author

Wes McKinney / @wesm:
Related; PARQUET-267. I will incorporate those changes (and other feedback from this PR) and report back.

@asfimport
Copy link
Collaborator Author

Wes McKinney / @wesm:
@nongli I opened a new PR apache/parquet-cpp#14. Will look at rebasing changes in PARQUET-267 on this PR based on what's decided re: build toolchain.

@asfimport
Copy link
Collaborator Author

Nong Li / @nongli:
Issue resolved by pull request 14
apache/parquet-cpp#14

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants