-
Notifications
You must be signed in to change notification settings - Fork 4k
ARROW-15027: [C++] Fix OpenTelemetry CMake definitions #11903
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
Conversation
|
The Gandiva failures seem unrelated. |
|
I'll file an issue for Gandiva since it appears on master. It seems to have started with b31dd51 even though that looks unrelated; since the failing builds use unity builds it seems something about it shifted how the unity build combines files. |
|
Thanks. I'll file a PR for it. |
|
I filed ARROW-15028 |
|
I filed #11905 -- hopefully it'll pass. |
|
The Gandiva fix was merged, it's in git master now. |
|
Looks like this passes once rebased, thanks for fixing the issue. |
|
Benchmark runs are scheduled for baseline = afa5d95 and contender = cba23c4. cba23c4 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
No description provided.