From eb12ef1dcd50aa5f58e0727831650a757e86488f Mon Sep 17 00:00:00 2001 From: "tom.bescherer" Date: Thu, 6 Jun 2019 16:40:32 -0400 Subject: [PATCH 1/2] move target_model vars inside loop to avoid reuse on subsequent refs --- core/dbt/parser/util.py | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/core/dbt/parser/util.py b/core/dbt/parser/util.py index e05c13f3f7d..2d9b2c146b4 100644 --- a/core/dbt/parser/util.py +++ b/core/dbt/parser/util.py @@ -173,11 +173,10 @@ def process_docs(cls, manifest, current_project): @classmethod def process_refs_for_node(cls, manifest, current_project, node): """Given a manifest and a node in that manifest, process its refs""" - target_model = None - target_model_name = None - target_model_package = None - for ref in node.refs: + target_model = None + target_model_name = None + target_model_package = None if len(ref) == 1: target_model_name = ref[0] elif len(ref) == 2: From ca31b79cc007a264d54b915a78f14879d0652825 Mon Sep 17 00:00:00 2001 From: "tom.bescherer" Date: Fri, 7 Jun 2019 11:57:19 -0400 Subject: [PATCH 2/2] add a newline for readability --- core/dbt/parser/util.py | 1 + 1 file changed, 1 insertion(+) diff --git a/core/dbt/parser/util.py b/core/dbt/parser/util.py index 2d9b2c146b4..a17a7f87bd9 100644 --- a/core/dbt/parser/util.py +++ b/core/dbt/parser/util.py @@ -177,6 +177,7 @@ def process_refs_for_node(cls, manifest, current_project, node): target_model = None target_model_name = None target_model_package = None + if len(ref) == 1: target_model_name = ref[0] elif len(ref) == 2: