Skip to content

Fix!: sqlmesh.dbt.adapter.RuntimeAdapter.get_columns_in_relation()#5115

Merged
newtonapple merged 3 commits intomainfrom
ddai/issue-1874
Aug 13, 2025
Merged

Fix!: sqlmesh.dbt.adapter.RuntimeAdapter.get_columns_in_relation()#5115
newtonapple merged 3 commits intomainfrom
ddai/issue-1874

Conversation

@newtonapple
Copy link
Contributor

returns columns in the proper Column subclass type.

fixes #1874

@newtonapple newtonapple requested a review from tobymao August 8, 2025 08:33
@izeigerman
Copy link
Collaborator

Thanks for clarifying. This looks good. How do we test this?

@newtonapple
Copy link
Contributor Author

Thanks for clarifying. This looks good. How do we test this?

I added an integration test for get_bq_schema(). But, I'm not sure where is the best place to test the dbt adapter. Should I create a test for RuntimeAdapter.get_columns_in_relation() in the bigquey adapter integration?

@newtonapple
Copy link
Contributor Author

How do we test this?

Thanks for clarifying. This looks good. How do we test this?

Thanks for clarifying. This looks good. How do we test this?

I added an integration test for get_bq_schema(). But, I'm not sure where is the best place to test the dbt adapter. Should I create a test for RuntimeAdapter.get_columns_in_relation() in the bigquey adapter integration?

Actually, I think I can mock it out.

@newtonapple newtonapple marked this pull request as ready for review August 11, 2025 22:05
@newtonapple newtonapple merged commit 0c70406 into main Aug 13, 2025
28 checks passed
@newtonapple newtonapple deleted the ddai/issue-1874 branch August 13, 2025 23:38
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.

dbt get_column_in_relation should return proper subclass

2 participants