[ADAP-620] [Bug] Incremental materialization with "append" strategy d…

…oes not wrap dml with explicit transaction (#673) * [ADAP-620] [Bug] Incremental materialization with "append" strategy does not wrap dml with explicit transaction Fixes #656 * changie --------- Co-authored-by: Mike Alfare <[email protected]>
dbt-labs · Aug 3, 2023 · 8423ed0 · 8423ed0
1 parent 5489398
commit 8423ed0
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/.changes/unreleased/Fixes-20230626-132631.yaml b/.changes/unreleased/Fixes-20230626-132631.yaml
@@ -0,0 +1,7 @@
+kind: Fixes
+body: Incremental materialization with "append" strategy does not wrap dml with explicit
+  transaction
+time: 2023-06-26T13:26:31.575359809Z
+custom:
+  Author: danielefrigo
+  Issue: "656"
diff --git a/dbt/include/snowflake/macros/materializations/merge.sql b/dbt/include/snowflake/macros/materializations/merge.sql
@@ -42,3 +42,9 @@
     {% set dml = default__snapshot_merge_sql(target, source, insert_cols) %}
     {% do return(snowflake_dml_explicit_transaction(dml)) %}
 {% endmacro %}
+
+
+{% macro snowflake__get_incremental_append_sql(get_incremental_append_sql) %}
+    {% set dml = default__get_incremental_append_sql(get_incremental_append_sql) %}
+    {% do return(snowflake_dml_explicit_transaction(dml)) %}
+{% endmacro %}