redefine type_string for redshift for dbt_project_evaluator's models only #473
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.
Creates a macro that redefine type_string for models in dbt_project_evaluator package. No need to override it for all models in a project anymore when using package.
This is a:
Link to Issue
Closes #469
Description & motivation
There are currently two ways database specific types are redefined for models in this package.
One is by creating a custom macro which is referenced with a dbt_project_evaluator prefix whenever used. See type_large_string.sql and usage example in base_node_columns.sql
The other is by overriding type definition globally for a project which causes all existing models to be changed if they use this type definition.
In my opinion the first way is better because any changes are limited to models in dbt_project_evaluator, that's why I propose this PR.
Integration Test Screenshot
Checklist
I tested that on a local version of this package without dispatcher configuration.