Check resource_type before DbtReferenceError

dbt-labs · Jun 9, 2023 · 108a0a7 · 108a0a7
1 parent 21e08d8
commit 108a0a7
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/core/dbt/context/providers.py b/core/dbt/context/providers.py
@@ -502,6 +502,9 @@ def resolve(
         elif (
             target_model.resource_type == NodeType.Model
             and target_model.access == AccessType.Private
+            # don't raise this reference error for ad hoc 'preview' queries
+            and self.model.resource_type != NodeType.SqlOperation
+            and self.model.resource_type != NodeType.RPCCall  # TODO: rm
         ):
             if not self.model.group or self.model.group != target_model.group:
                 raise DbtReferenceError(

diff --git a/core/dbt/parser/manifest.py b/core/dbt/parser/manifest.py
@@ -1683,8 +1683,13 @@ def _process_refs_for_node(manifest: Manifest, current_project: str, node: Manif
             )
             continue
 
-        # Handle references to models that are private
-        elif isinstance(target_model, ModelNode) and target_model.access == AccessType.Private:
+        # Handle references to models that are private, unless this is an 'ad hoc' query (SqlOperation, RPCCall)
+        elif (
+            isinstance(target_model, ModelNode)
+            and target_model.access == AccessType.Private
+            and node.resource_type != NodeType.SqlOperation
+            and node.resource_type != NodeType.RPCCall  # TODO: rm
+        ):
             if not node.group or node.group != target_model.group:
                 raise dbt.exceptions.DbtReferenceError(
                     unique_id=node.unique_id,