Skip to content

Commit 8ee37f6

Browse files
committed
fix
1 parent 37c2cf2 commit 8ee37f6

1 file changed

Lines changed: 13 additions & 11 deletions

File tree

  • dbt/include/databricks/macros/materializations/incremental

dbt/include/databricks/macros/materializations/incremental/strategies.sql

Lines changed: 13 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,9 @@
2525
{{ return(get_insert_overwrite_sql(source, target)) }}
2626
{% endmacro %}
2727

28-
{% macro get_common_columns(dest_columns, source_columns) %}
28+
{% macro get_insert_overwrite_sql(source_relation, target_relation) %}
29+
{%- set dest_columns = adapter.get_columns_in_relation(target_relation) | map(attribute='quoted') | list -%}
30+
{%- set source_columns = adapter.get_columns_in_relation(source_relation) | map(attribute='quoted') | list -%}
2931
{%- set common_columns = [] -%}
3032
{%- for dest_col in dest_columns -%}
3133
{%- if dest_col in source_columns -%}
@@ -34,13 +36,6 @@
3436
{%- do common_columns.append('DEFAULT') -%}
3537
{%- endif -%}
3638
{%- endfor -%}
37-
{{ return(common_columns) }}
38-
{% endmacro %}
39-
40-
{% macro get_insert_overwrite_sql(source_relation, target_relation) %}
41-
{%- set dest_columns = adapter.get_columns_in_relation(target_relation) | map(attribute='quoted') | list -%}
42-
{%- set source_columns = adapter.get_columns_in_relation(source_relation) | map(attribute='quoted') | list -%}
43-
{%- set common_columns = get_common_columns(dest_columns, source_columns) -%}
4439
{%- set dest_cols_csv = dest_columns | join(', ') -%}
4540
{%- set source_cols_csv = common_columns | join(', ') -%}
4641
insert overwrite table {{ target_relation }}
@@ -70,11 +65,18 @@ TABLE {{ temp_relation.render() }}
7065
{% endmacro %}
7166

7267
{% macro insert_into_sql_impl(target_relation, dest_columns, source_relation, source_columns) %}
73-
{%- set common_columns = get_common_columns(dest_columns, source_columns) -%}
68+
{%- set common_columns = [] -%}
69+
{%- for dest_col in dest_columns -%}
70+
{%- if dest_col in source_columns -%}
71+
{%- do common_columns.append(dest_col) -%}
72+
{%- else -%}
73+
{%- do common_columns.append('DEFAULT') -%}
74+
{%- endif -%}
75+
{%- endfor -%}
7476
{%- set dest_cols_csv = dest_columns | join(', ') -%}
7577
{%- set source_cols_csv = common_columns | join(', ') -%}
76-
insert into table {{ target_relation }} ({{ dest_cols_csv }})
77-
select {{source_cols_csv}} from {{ source_relation }}
78+
insert into table {{ target_relation }} ({{ dest_cols_csv }})
79+
select {{source_cols_csv}} from {{ source_relation }}
7880
{%- endmacro %}
7981

8082
{% macro databricks__get_merge_sql(target, source, unique_key, dest_columns, incremental_predicates) %}

0 commit comments

Comments
 (0)