databricks / dbt-databricks

A dbt adapter for Databricks.
https://databricks.com
Apache License 2.0
226 stars 119 forks source link

Stop setting cluster by to None #806

Closed benc-db closed 1 month ago

benc-db commented 1 month ago

Resolves #805

Description

This fixes 805, though 805 raises a bigger question about whether its reasonable for users to materialize things outside of dbt, and expect dbt to handle that gracefully if you don't add matching config to the project. Setting that aside, actually getting rid of this clause because I think it's more reasonable for someone to do a full refresh if they want to get rid of liquid clustering than it is to run cluster by none on every Delta incremental run.

Checklist