Merge pull request #1608 from fishtown-analytics/fix/is-incremental-c…

…heck-materialization Make is_incremental live up to its name
dbt-labs · Jul 15, 2019 · 5a1f0bd · 5a1f0bd
2 parents cc90b04 + 405748c
commit 5a1f0bd
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 1 deletion.
diff --git a/core/dbt/include/global_project/macros/etc/is_incremental.sql b/core/dbt/include/global_project/macros/etc/is_incremental.sql
@@ -5,6 +5,9 @@
         {{ return(False) }}
     {% else %}
         {% set relation = adapter.get_relation(this.database, this.schema, this.table) %}
-        {{ return(relation is not none and relation.type == 'table' and not flags.FULL_REFRESH) }}
+        {{ return(relation is not none
+                  and relation.type == 'table'
+                  and model.config.materialized == 'incremental'
+                  and not flags.FULL_REFRESH) }}
     {% endif %}
 {% endmacro %}
diff --git a/test/integration/017_runtime_materialization_tests/models/materialized.sql b/test/integration/017_runtime_materialization_tests/models/materialized.sql
@@ -5,3 +5,7 @@
 }}
 
 select * from {{ this.schema }}.seed
+
+{% if is_incremental() %}
+    {% do exceptions.raise_compiler_error("is_incremental() evaluated to True in a table") %}
+{% endif %}
diff --git a/test/integration/017_runtime_materialization_tests/models/view.sql b/test/integration/017_runtime_materialization_tests/models/view.sql
@@ -5,3 +5,7 @@
 }}
 
 select * from {{ this.schema }}.seed
+
+{% if is_incremental() %}
+    {% do exceptions.raise_compiler_error("is_incremental() evaluated to True in a view") %}
+{% endif %}