You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This is hard-coded within dbt-core right now, which doesn't account for:
new materializations, e.g. materialized_view / dynamic_table
custom materializations (e.g. jercos_special_table) that might totally enforce constraints!
But it will be tricky because we're raising the warning at parsing time right now.
Two proposed options:
Move this check to execution time, and make it an adapter-overridable method (similar to adapter enforced/support checks for specific constraint types) or user-overridable macro
Turn this into a "negative" check, and raise it if a user has configured constraints on a view. If they're using a custom materialization, that's on them.
Is this your first time submitting a feature request?
Describe the feature
dbt-core/core/dbt/contracts/graph/nodes.py
Lines 479 to 481 in 341803d
This is hard-coded within
dbt-core
right now, which doesn't account for:materialized_view
/dynamic_table
jercos_special_table
) that might totally enforce constraints!But it will be tricky because we're raising the warning at parsing time right now.
Two proposed options:
view
. If they're using a custom materialization, that's on them.Describe alternatives you've considered
No response
Who will this benefit?
Are you interested in contributing this feature?
No response
Anything else?
No response
The text was updated successfully, but these errors were encountered: