Add object to context with compiled code for full refresh #11383
+17
−0
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Part of the solution to resolve dbt-labs/dbt-adapters#154
Needs to be released along with dbt-labs/dbt-adapters#900 for the materialization code.
Problem
Today it is not possible to set
on_schema_change = full_refresh
for incremental models.Solution
To make it possible to add the feature, this implementation adds an object called
sql_full_refresh
to the context so that the materialization macros indbt-adapters
can use this variable.This prevents having to re-compile the model if we realize that a model should run as a full refresh when its schema changes.
There is no test for now, but I am happy to add some if the approach in this PR along with the dbt-adapters one is OK to be merged.
Checklist