Fix for: #819 (#820)

Co-authored-by: Ben Cassell <98852248+benc-db@users.noreply.github.com>
databricks · Oct 4, 2024 · fa5f89e · fa5f89e
1 parent d2f3f82
commit fa5f89e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -6,7 +6,8 @@
 
 ### Fixes
 
-- Stop setting cluster by to None. If you want to drop liquid clustering, you will need to full-refresh ([806](https://github.com/databricks/dbt-databricks/pull/806))
+- Stop setting cluster by to None. If you want to drop liquid clustering, you will need to full-refresh ([806]https://github.com/databricks/dbt-databricks/pull/806)
+- Don't define table properties on snapshot staging views (thanks @jelmerk!) ([820](https://github.com/databricks/dbt-databricks/pull/820))
 
 ## dbt-databricks 1.8.6 (September 18, 2024)
 

diff --git a/dbt/include/databricks/macros/materializations/snapshot.sql b/dbt/include/databricks/macros/materializations/snapshot.sql
@@ -10,7 +10,9 @@
 
     {# needs to be a non-temp view so that its columns can be ascertained via `describe` #}
     {% call statement('build_snapshot_staging_relation') %}
-        {{ create_view_as(tmp_relation, select) }}
+        create or replace view {{ tmp_relation }}
+        as
+            {{ select }}
     {% endcall %}
 
     {% do return(tmp_relation) %}