Skip to content

Conversation

benc-db
Copy link
Collaborator

@benc-db benc-db commented Sep 22, 2025

Description

Customer raised issue where merge was specifying DEFAULT for every column, but it should not have even called out the columns because they hadn't changed. Looks like the issue was related to capitalization issues.

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.

@benc-db benc-db requested a review from ericj-db as a code owner September 22, 2025 22:55
@benc-db benc-db changed the base branch from main to 1.11.latest September 22, 2025 22:56
@benc-db benc-db changed the title Ensure lower case column compare fix: Ensure lower case column compare Sep 22, 2025
Copy link

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  dbt/adapters/databricks
  impl.py 788-789, 818-833
  dbt/adapters/databricks/relation_configs
  column_comments.py 20-26
  column_mask.py
Project Total  

This report was generated by python-coverage-comment-action

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.

1 participant