update_parition_by_macro

Author: fivetran-savage

CHANGELOG.md

@@ -1,11 +1,10 @@
 # dbt_quickbooks v1.4.2
 
-[PR #198](https://github.com/fivetran/dbt_quickbooks/pull/198) includes the following updates:
+[PR #198](https://github.com/fivetran/dbt_quickbooks/pull/198) includes the following update:
 
 ## Under the Hood
 - Introduces the `partition_by_source_relation` macro to conditionally include `source_relation` in window function PARTITION BY clauses only when multiple sources are configured via the `quickbooks_sources` variable. Replaces hardcoded `source_relation` partitioning in the following models: `int_quickbooks__bill_payment_double_entry`, `int_quickbooks__credit_card_pymt_double_entry`, `int_quickbooks__payment_double_entry`, `int_quickbooks__transfer_double_entry`, `int_quickbooks__general_ledger_balances`, `quickbooks__cash_flow_statement`, `quickbooks__general_ledger`, and `stg_quickbooks__credit_card_payment_txn`.
 
-
 # dbt_quickbooks v1.4.1
 
 [PR #196](https://github.com/fivetran/dbt_quickbooks/pull/196) includes the following updates:

macros/staging/partition_by_source_relation.sql

@@ -1,15 +1,15 @@
-{%- macro partition_by_source_relation(has_other_partitions='yes', alias=None) -%}
+{% macro partition_by_source_relation(has_other_partitions='yes', alias=None) %}
+    {{ return(adapter.dispatch('partition_by_source_relation', 'quickbooks')(has_other_partitions, alias)) }}
+{% endmacro %}
 
-{{ adapter.dispatch('partition_by_source_relation', 'quickbooks') (has_other_partitions, alias) }}
+{% macro default__partition_by_source_relation(has_other_partitions='yes', alias=None) -%}
 
-{%- endmacro %}
+{%- set prefix = '' if alias is none else alias ~ '.' -%}
 
-{% macro default__partition_by_source_relation(has_other_partitions='yes', alias=None) -%}
-    {% set prefix = '' if alias is none else alias ~ '.' %}
-
-    {%- if has_other_partitions == 'no' -%}
-        {{ 'partition by ' ~ prefix ~ 'source_relation' if var('quickbooks_sources', [])|length > 1 }}
-    {%- else -%}
-        {{ ', ' ~ prefix ~ 'source_relation' if var('quickbooks_sources', [])|length > 1 }}
-    {%- endif -%}
-{%- endmacro -%}
+{%- if has_other_partitions == 'no' -%}
+    {{- 'partition by ' ~ prefix ~ 'source_relation' if var('quickbooks_union_schemas', [])|length > 1 or var('quickbooks_union_databases', [])|length > 1 else '' -}}
+{%- else -%}
+    {{- ', ' ~ prefix ~ 'source_relation' if var('quickbooks_union_schemas', [])|length > 1 or var('quickbooks_union_databases', [])|length > 1 else '' -}}
+{%- endif -%}
+
+{%- endmacro %}