add docs for @unique directive in schema #638
Closed
Conversation
This file contains 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
mangalaman93
requested review from
rderbier,
MichelDiz,
damonfeldman,
rarvikar and
Rajakavitha1
as code owners
4 tasks
✅ Deploy Preview for dgraph-docs-repo ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
mangalaman93
added a commit
to dgraph-io/dgraph
that referenced
this pull request
Aug 28, 2023
#8827) Partially Fixes #8827 Closes: DGRAPHCORE-206 Docs PR: dgraph-io/dgraph-docs#638 This PR adds support for uniqueness constraint using @unique directive in DQL schema. This unique directive ensures that all values of the predicate are different in a Dgraph Cluster. This completes phase 1, and enables adding a new predicate with unique directive. As part of the phase 2, we will work on adding support for unique directive for existing predicates. ## Performance Live Loader before this change on 21 million dataset took 10m54s whereas after this change took 11m02s. It did not make any significant different to non-unique predicates. ## How to Use You can now specify unique in schema as follows: `email: string @unique @Index(hash) @upsert .`. Now, Dgraph will ensure that no mutation adds a duplicate for the predicate email. ## Phase 2 [TODO] - [ ] check if @unique can be added to schema depending upon whether existing data has any duplicates. If the existing data has any duplicates, we do not allow adding the @unique directive and return a query that allows user to identify these UIDs. - [ ] If index computation is in progress, we should not allow mutations with predicates for which @unique is set - [ ] Fix ACL to ensure that we do not end up adding duplicate users - [ ] Ensure that unique constraint is not violated during Bulk loader --------- Co-authored-by: Aman Mangal <aman@dgraph.io>
|
Hi @mangalaman93 can we please change the status of this PR to draft |
|
closing in lieu of #644 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.