-
Notifications
You must be signed in to change notification settings - Fork 943
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update constraints.md #3736
Update constraints.md #3736
Conversation
✅ Deploy Preview for docs-getdbt-com ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
}} | ||
|
||
select | ||
'string' as a, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be nice if this example were more "real world" but this gets the point across.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added one comment for stylistic consistency with the contract docs example
version: 2 | ||
|
||
models: | ||
- name: double_nested_fields |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: this name doesn't match the file name above: nested_column_constraints_example
- name: double_nested_fields | ||
config: | ||
contract: {enforced: true} | ||
materialized: table |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: don't think this is necessary since we have config(materialized='table')
in the sql contents already
Co-authored-by: Grace Goheen <[email protected]>
resolves #3454
What are you changing in this pull request and why?
Adding an example of column level constraints for BigQuery only.
Checklist
Adding new pages (delete if not applicable):
website/sidebars.js
Removing or renaming existing pages (delete if not applicable):
website/sidebars.js
website/static/_redirects