-
Notifications
You must be signed in to change notification settings - Fork 173
[WIP] Backport external table clone on top of 1.9.latest #1189
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
base: 1.9.latest
Are you sure you want to change the base?
[WIP] Backport external table clone on top of 1.9.latest #1189
Conversation
…pport (not tested) Signed-off-by: Dmitry Volodin <volodin.d.v@gmail.com>
|
@samgans please review. I am fixing the conflicts as 1.9.latest was updated |
| DBT_DATABRICKS_HTTP_SESSION_HEADERS, | ||
| DBT_DATABRICKS_INVOCATION_ENV, |
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.
Why is it there? I don't see any mentions of those in the code.
Co-authored-by: samgans <63362199+samgans@users.noreply.github.com>
| from typing import Optional | ||
|
|
||
| from dbt.adapters.databricks import catalogs, constants |
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.
@samgans I think all the changes in this files is unrelated to your code, is that correct?
|
We put this PR on pause. The driver was broken elementary, but they've announced that they fix the compatibility with dbt-databricks soon. They've already released 0.20 with version now being non-pinned |
This is a backport of the changes introduced in #1079 and #1095.
Not everything is applied cleanly as there are some major changes between 1.9 and 1.10 in terms of metadata management and testing.
Description
Checklist
CHANGELOG.mdand added information about my change to the "dbt-databricks next" section.