-
Notifications
You must be signed in to change notification settings - Fork 4k
ARROW-16102: [C++] Add support for building with system gRPC and bundled GCS #12795
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
f7686e4 to
b030442
Compare
|
@emkornfield Do you want to merge this before #12763 is merged or after? This will be conflicted with #12763. This will be ready to merge once https://github.com/apache/arrow/runs/5864515682?check_suite_focus=true is passed. |
|
@github-actions crossbow submit -g cpp |
|
Revision: bffc9dcdf7df9bfa8ae46898cd895adf5355016e Submitted crossbow builds: ursacomputing/crossbow @ actions-1823 |
|
@emkornfield OK. Thanks for confirming it. |
|
@github-actions crossbow submit -g cpp |
|
Revision: 9685ef52fa2d2559cf3a47b2187308d43359e997 Submitted crossbow builds: ursacomputing/crossbow @ actions-1831 |
05798f8 to
3e279ab
Compare
|
@github-actions crossbow submit -g cpp |
|
Revision: 3e279ab Submitted crossbow builds: ursacomputing/crossbow @ actions-1832 |
|
CI failures are unrelated. This is ready to merge. |
|
+1 |
|
Benchmark runs are scheduled for baseline = 29f3873 and contender = 5fa0844. 5fa0844 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
find_package(gRPC)may find systemabsl::*targets.If we found system
absl::*targets, we use them instead of buildingabsl::*.Bundle google-cloud-cpp is updated to 1.39.0 for
googleapis/google-cloud-cpp#8536 .