-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Support BigQuery client or URI in database I/O functions #18547
Comments
@alexander-beedie - I was mid way through drafting this when I saw your comment on #17326, so I figured I'd create it now. Would be keen to get your thoughts on which approach you prefer. I would also be keen to work on this or help out with an implementation. |
I'll poke at it for a bit and see what appears to work best; my feeling is we likely want to accept the BigQuery Client object itself in Luckily I have access to a decent-sized BigQuery instance, so I should be able to try a few things out in the near-ish future! At that point some help testing, validating, improving, etc would definitely be appreciated ✌️ |
Have filled out the rest of the issue now 😄
Cool. The URI option also seems viable for both read and write. Saves the user instantiating the client if all they are going to do is pass it to the function. It can be created for them. Obviously if the user wants more control over how to authenticate or which project to bill, they can provide the client. So, perhaps both could be supported? This would be similar to how I can a give a Postgres URI to
Sounds good! And appreciate the prompt response 😀 |
Yup, could make sense 👌 |
Description
At the moment writing to BigQuery requires some code that is not all that obvious, and differs significantly from every other supported database/driver. Reading data is more obvious, but could also be simplified further. It would be really nice if BigQuery could fit a bit more nicely into the existing Polars read/write database functions.
Below are some proposed options of what this could look like, compared against existing working solutions.
The text was updated successfully, but these errors were encountered: