(fixes #1119) Ignore dependencies into dbt schemas from external schemas

dbt/adapters/postgres/impl.py

@@ -158,7 +158,10 @@ def _link_cached_relations(self, manifest, schemas):
                                               identifier=refed_name)
             dependent = self.Relation.create(schema=dep_schema,
                                              identifier=dep_name)
-            self.cache.add_link(dependent, referenced)
+
+            # don't record in cache if this relation isn't in a relevant schema
+            if refed_schema in schemas:
+                self.cache.add_link(dependent, referenced)
 
     def _list_relations(self, schema, model_name=None):
         sql = """

test/integration/037_external_reference_test/standalone_models/my_model.sql

@@ -0,0 +1,2 @@
+
+select 1 as id

test/integration/037_external_reference_test/test_external_reference.py

@@ -37,3 +37,39 @@ def test__postgres__external_reference(self):
         self.assertEquals(len(self.run_dbt()), 1)
         # running it again should succeed
         self.assertEquals(len(self.run_dbt()), 1)
+
+# The opposite of the test above -- check that external relations that
+# depend on a dbt model do not create issues with caching
+class TestExternalDependency(DBTIntegrationTest):
+    @property
+    def schema(self):
+        return "external_dependency_037"
+
+    @property
+    def models(self):
+        return "test/integration/037_external_reference_test/standalone_models"
+
+    def tearDown(self):
+        # This has to happen before we drop the external schema, because
+        # otherwise postgres hangs forever.
+        self._drop_schema()
+        self.adapter.drop_schema(self.external_schema, '__test')
+        super(TestExternalDependency, self).tearDown()
+
+    @use_profile('postgres')
+    def test__postgres__external_reference(self):
+        self.assertEquals(len(self.run_dbt()), 1)
+
+        # create a view outside of the dbt schema that depends on this model
+        self.external_schema = self.unique_schema()+'zz'
+        self.run_sql(
+            'create schema "{}"'.format(self.external_schema)
+        )
+        self.run_sql(
+            'create view "{}"."external" as (select * from {}.my_model)'
+            .format(self.external_schema, self.unique_schema())
+        )
+
+        # running it again should succeed
+        self.assertEquals(len(self.run_dbt()), 1)
+