Merge pull request #296 from dbt-labs/fix/issue-ducplicates-v4

🐛 Fix for duplicate PKs in stg relationships

integration_tests/models/marts/exposures.yml

@@ -14,4 +14,5 @@ exposures:
     depends_on:
       - ref('fct_model_6')
       - ref('dim_model_7')
+      - ref('dim_model_7')
       - source('source_1', 'table_1')

integration_tests/models/marts/int_model_4.sql

@@ -1,4 +1,6 @@
 -- depends on: {{ ref('stg_model_1') }}
+-- depends on: {{ ref('stg_model_1') }}
+-- depends on: {{ ref('stg_model_1') }}
 -- depends on: {{ source('source_1', 'table_2') }}
 
 select 1 as id

macros/unpack/get_relationship_values.sql

@@ -39,7 +39,7 @@
                         [
                             "cast('" ~ node.unique_id ~ "' as " ~ dbt.type_string() ~ ")",
                             "cast('" ~ parent ~ "' as " ~ dbt.type_string() ~ ")",
-                            "" ~ loop.last ~ ""
+                            "" ~ loop.last ~ "" if node.unique_id.split('.')[0] == 'test' else "FALSE"
                         ]
                     %}
 

models/staging/graph/stg_metric_relationships.sql

@@ -11,4 +11,4 @@ final as (
     from _base_metric_relationships
 )
 
-select * from final
+select distinct * from final

models/staging/graph/stg_node_relationships.sql

@@ -11,4 +11,5 @@ final as (
     from _base_node_relationships
 )
 
-select  * from final
+-- we need distinct as the graph lists relationships multiple times if they are ref'd multiple times
+select distinct * from final