-
Notifications
You must be signed in to change notification settings - Fork 601
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
BigQuery cloud test for Iceberg catalogs #24969
base: dev
Are you sure you want to change the base?
Conversation
fb5ab7c
to
9906648
Compare
But this is NOT a blocker for this PR. We should be able to review and merge it as tests work As a follow up, we could extend it with other supported modes, more topics, different data formats, etc. But those enhancements are NOT part of this PR. |
9906648
to
b022ea6
Compare
@travisdowns, Adding you as a reviewer as well, since you're familiar with cloud tests and this should make sense to you. We can go over details of this whenever you have time. |
storage_uri_prefix = 'gs' | ||
if cloud_provider == "aws": | ||
storage_uri_prefix = 's3' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't work on Azure? Or just hasn't been tested there yet? (Also does Bigquery work on AWS? I thought it was GCP-only)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It should work on Azure, but requires cross cloud policy setup, which is not yet ready and not tested with Azure storage, but in theory should work the same. We can add Azure storage once this gets merged and tested on other clouds.
Bigquery itself is on GCP, but when we create Datasets and external tables, it could query "cross-cloud". Details are documented in the Conflucne page that I shared. Requres one-time setup to allow BigQuery to connect to Redpanda storage buckets on another cloud
Summary:
bq
tool on GCP and AWS clustersBackports Required
Release Notes