-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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 database driver #881
base: master
Are you sure you want to change the base?
Conversation
Hey, I was comparing against #830 and I was just curious if you were going to pick this back up. |
Hi. I haven't found a stable emulator for the bigquery. That's why there are a lot of mistakes in testing. BTW, it is working with real data. |
Oh, have you tried https://github.com/goccy/bigquery-emulator ? I used it for a handful of tests and it worked reasonably well there. The DDL support is mostly |
Unfortunately bigquery-emulator didn't allow to use system variables. We are using it to set automaticaly '@@dataset_id' in migrations scripts. I have create issue for this case (goccy/bigquery-emulator#147). And alse the problem is that dktest does not allow to open a custom port. I created a pull request (dhui/dktest#21) in order to allow opening the port manually, but so far no one has approved it. |
5d4473d
to
a0b7408
Compare
/cc @dhui |
I'm not sure what's happening with the tests. It seems to be an issue with sqlserver. However, these changes are unrelated to that. I also tried to run it on master and I have the same issue. |
Created an issue #1174. |
Add support for BigQuery database driver.