Skip to content
This repository was archived by the owner on May 17, 2024. It is now read-only.

allow joindiff across bigquery projects #348

Merged
merged 1 commit into from
Dec 13, 2022
Merged

Conversation

segv
Copy link
Contributor

@segv segv commented Dec 12, 2022

No description provided.

@erezsh
Copy link
Contributor

erezsh commented Dec 12, 2022

Why not just follow the simple way we implemented it for snowflake, where you use the same database connection, but then specify a different project in the table path?

@segv
Copy link
Contributor Author

segv commented Dec 13, 2022

Why not just follow the simple way we implemented it for snowflake, where you use the same database connection, but then specify a different project in the table path?

My brain just didn't make the connection that this was what you were talking about. PR updated.

@erezsh
Copy link
Contributor

erezsh commented Dec 13, 2022

Looking better! I believe the change in info_tree.py isn't necessary anymore, right?

@erezsh
Copy link
Contributor

erezsh commented Dec 13, 2022

(don't worry about the CI error, it's a problem on our end)

@segv
Copy link
Contributor Author

segv commented Dec 13, 2022

I reverted the rowcounts: Dict[int, int] = {1: 0, 2: 0} change.

erezsh added a commit that referenced this pull request Dec 13, 2022
@erezsh erezsh merged commit d381e99 into datafold:master Dec 13, 2022
@erezsh
Copy link
Contributor

erezsh commented Dec 13, 2022

Thank you for contributing!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants