Skip to content
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.

Sign up for GitHub

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

Jump to bottom

DROP SOURCE can no longer be used to drop subsources, breaks dbt #20483

Closed
philip-stoev opened this issue Jul 12, 2023 · 0 comments · Fixed by #20501
Closed

DROP SOURCE can no longer be used to drop subsources, breaks dbt #20483

philip-stoev opened this issue Jul 12, 2023 · 0 comments · Fixed by #20501
Assignees
@morsapaes
Labels
C-bug Category: something is broken T-dbt Theme: dbt adapter

Comments

@philip-stoev
Copy link
Contributor

philip-stoev commented Jul 12, 2023 •
edited

What version of Materialize are you using?

v0.61.0-dev (dcdb2bd)

What is the issue?

dbt can no longer drop subsources when it wants to recreate them:

13:37:41  Database Error in model supplier (models/loadgen/supplier.sql)
13:37:41    SOURCE "supplier" is a subsource and must be dropped with ALTER SOURCE...DROP SUBSOURCE
13:37:41    HINT:  Use ALTER SOURCE tpch DROP SUBSOURCE supplier

A test can be made available if needed.

@sploiselle FYI
@philip-stoev philip-stoev added the C-bug Category: something is broken label Jul 12, 2023
@philip-stoev philip-stoev added the T-dbt Theme: dbt adapter label Jul 12, 2023
@philip-stoev philip-stoev changed the title DROP SOURCE can no longer be used to drop Postgres subsources, breaks dbt DROP SOURCE can no longer be used to drop subsources, breaks dbt Jul 12, 2023
@chaas chaas assigned sploiselle and unassigned chaas Jul 12, 2023
@morsapaes morsapaes assigned morsapaes and unassigned sploiselle Jul 13, 2023
@morsapaes morsapaes mentioned this issue Jul 13, 2023
Merged
morsapaes added a commit that referenced this issue Jul 13, 2023
@morsapaes
dbt-materialize: exclude subsources from introspection (#20501)
5d7efdf 
Fixes #20483.
@morsapaes morsapaes mentioned this issue Jul 13, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@morsapaes morsapaes

Labels
C-bug Category: something is broken T-dbt Theme: dbt adapter
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

dbt-materialize: exclude subsources from introspection morsapaes/materialize
4 participants
@philip-stoev @chaas @sploiselle @morsapaes