GH-46410: [C++] Add parquet options to Meson configuration
Rationale for this change
This continues adding functionality to the Meson configuration
What changes are included in this PR?
This adds the parquet directory to the Meson configuration
Are these changes tested?
Yes
Are there any user-facing changes?
No
- GitHub Issue: #46410
:warning: GitHub issue #46410 has been automatically assigned in GitHub to PR creator.
@kou this one might have gotten missed - mind taking a look when you have time?
Hey @kou just wanted to toss a friendly ping on this. If you have time, please let me know what you think of the latest updates. Thanks!
Looks like the current failures have to do with symbol visibility issues with template instantiations, which I don't believe the CMake configuration is testing. A simple workaround would be to remove the hidden symbol visibility from the Meson configuration, although I'd ideally like to resolve instead
The Alpine Linux failure is related to https://github.com/apache/arrow/issues/47052 not this PR, and I've seen the rhub failure on other PRs as well, so I think unrelated
OK all green here, sans one expected failure.
This became bigger than originally anticipated because I wanted to keep the consistency of the Meson libraries using hidden symbol visibility, but that in turn required some changes to the parquet sources and CMake configuration. If easier for review, I'd be happy to try and split this into two different changes
Hey @kou let me know what you think about this. As mentioned, I am happy to split up some of the symbol fixing issues into a separate PR if that makes things easier
Sorry for not responding this...
Let's split C++/CMake changes to a separated PR for easy to review and track.
OK I've removed any of the symbol changes. Let me know what you think @kou - thanks!
I think all feedback here has been addressed - let me know what you think @kou
After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit 05e64889fac700eda737e6fb7410382f0ae7219d.
There weren't enough matching historic benchmark results to make a call on whether there were regressions.
The full Conbench report has more details.