Skip to content

Conversation

@mi-volodin
Copy link
Contributor

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

  • I have run this code in development and it appears to resolve the stated issue
  • This PR includes tests, or tests are not required/relevant for this PR
  • I have updated the CHANGELOG.md and added information about my change to the "dbt-databricks next" section.

…pport (not tested)

Signed-off-by: Dmitry Volodin <volodin.d.v@gmail.com>
@mi-volodin mi-volodin changed the base branch from main to 1.9.latest September 20, 2025 09:18
@mi-volodin
Copy link
Contributor Author

@samgans please review.

I am fixing the conflicts as 1.9.latest was updated

Comment on lines +14 to +15
DBT_DATABRICKS_HTTP_SESSION_HEADERS,
DBT_DATABRICKS_INVOCATION_ENV,
Copy link
Contributor

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>
Comment on lines +3 to +5
from typing import Optional

from dbt.adapters.databricks import catalogs, constants
Copy link
Contributor Author

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?

@mi-volodin
Copy link
Contributor Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants