ARROW-795: [C++] Consolidate arrow/arrow_io/arrow_ipc into a single shared and static library#516
ARROW-795: [C++] Consolidate arrow/arrow_io/arrow_ipc into a single shared and static library#516wesm wants to merge 2 commits intoapache:masterfrom
Conversation
…library Change-Id: Icdc7d231f59aceaa49f5699ea0cb4acb720590fa
|
@kou if you base changes for arrow-glib on top of this branch, can you post a git commit here for me to cherry pick into this PR, so that we can get a passing build? I can try my hand at cleaning things up if you like, too, but I wasn't sure about some of the generated code in arrow-glib/Makefile.am. Thanks! |
|
I've attached a patch: 0001-GLib-Consolidate-arrow-glib-arrow-io-glib-arrow-ipc-.txt |
Change-Id: Iac1c3cdce7d94fd2f49472bc511484d5e223f852
|
Thank you! I just applied the patch |
|
The test suite passed except for Parquet failures / S3 timeouts |
|
@pcmoritz @robertnishihara -- does this seem OK? The idea was to make Arrow simpler to develop against for thirdparties (and easier for us to package / develop internally). The public APIs still have the |
|
+1, @xhochy is on vacation and we spoke briefly about this offline, so I'm going to merge. If there's any unforeseen consequences I'll be responsive to make quick fixes |
|
This makes sense! I don't anticipate any problems here. |
This PR migrates the Arrow project to publish Build Scans to the the new Develocity instance at develocity.apache.org. Additionally, this PR sets a projectId for use by Develocity. Fixes apache#515.
3 participants
This leaves c_glib in a broken state, since I'd rather let @kou fix up things the way he prefers (I started refactoring the automake files and realized I would probably just make a mess).
I'm going to submit a patch to make parquet-cpp work on top of this.