Fabric+constraints - #9589
#9589Open
nataliefiann wants to merge 5 commits into
currentdbt-labs/docs.getdbt.com:currentfrom
nfiann-fabricconstraintsdbt-labs/docs.getdbt.com:nfiann-fabricconstraintsCopy head branch name to clipboard
Open
Fabric+constraints#9589nataliefiann wants to merge 5 commits intocurrentdbt-labs/docs.getdbt.com:currentfrom nfiann-fabricconstraintsdbt-labs/docs.getdbt.com:nfiann-fabricconstraintsCopy head branch name to clipboard
nataliefiann wants to merge 5 commits into
currentdbt-labs/docs.getdbt.com:currentfrom
nfiann-fabricconstraintsdbt-labs/docs.getdbt.com:nfiann-fabricconstraintsCopy head branch name to clipboard
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Looks very good to me. This was exactly the type of change I was looking for 😊 |
Co-authored-by: Matt Shaver <60105315+matthewshaver@users.noreply.github.com>
|
|
||
| Only `not_null` is checked when the model builds. Other supported constraint types are recorded on the table for reference, but Microsoft Fabric does not check them when new data is added. Use [data tests](/docs/build/data-tests) if you need to verify those rules. | ||
|
|
||
| :::note |
Contributor
There was a problem hiding this comment.
Do you feel the note is necessary given the table defines that it it not supported?
Contributor
Author
There was a problem hiding this comment.
Good point. I've removed the note. The table and paragraph already cover it.
The support table and paragraph already cover data tests as an alternative. Co-authored-by: Cursor <cursoragent@cursor.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
Resolves JIRA:PRODDOCS-1206
This PR adds Microsoft Fabric constraint documentation.
A customer on a Fabric PoC noted that Fabric was missing from the constraints page and felt under-documented compared to other adapters. The
dbt-fabricadapter supports constraints and model contracts (with limits), but the docs did not reflect that.This PR:
_constraints-table.md), which also updates the Model contracts pagecheckconstraintsdbt-fabricon the incremental strategy table (already documented infabric-configs.mdand supported in the adapter)Fabric constraint behavior documented:
not_nullis checked when the model buildsprimary_key,foreign_key, anduniqueare supported but not checked by Fabriccheckis not supported; use data tests insteadChecklist
🚀 Deployment available! Here are the direct links to the updated files: