Add ngram updates #189
Add ngram updates #189
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds support for the ngram search index type across Dgraph documentation, including comprehensive documentation of its functionality and usage patterns.
- Adds
ngramindex support to GraphQL and DQL documentation - Updates existing documentation for consistency and clarity improvements
- Provides detailed examples of ngram search functionality for contiguous sequence matching
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| dgraph/graphql/schema/directives/search.mdx | Adds ngram search documentation with examples and updates terminology consistency |
| dgraph/graphql/schema/dgraph-schema.mdx | Adds ngram to DgraphIndex enum |
| dgraph/dql/indexes.mdx | Documents ngram index in the string indexes table |
| dgraph/dql/functions.mdx | Adds comprehensive ngram function documentation with usage examples |
| dgraph/concepts/index-tokenize.mdx | Fixes spelling error in "Similarly" |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| @@ -85,15 +85,13 @@ contain the term "GraphQL". | |||
|
|
|||
| ```graphql | |||
| queryAuthor(filter: { name: { eq: "Diggy" } } ) { | |||
| posts(filter: { title: { anyofterms: "GraphQL" }}) { | |||
| title | |||
| } | |||
There was a problem hiding this comment.
The code block is incomplete - it's missing the closing brace and content. The removal of lines 88-89 appears to have left this query example broken.
Copilot uses AI. Check for mistakes.
|
|
||
| `fulltext` search is Google-stye text search with stop words, stemming. etc. So | ||
| `alloftext: "run woman"` would match "run" as well as "running", etc. For | ||
| example, to find posts that talk about fantastic GraphQL tutorials: |
There was a problem hiding this comment.
The removal of the explanation about allofterms vs anyofterms behavior (lines 284-286) leaves the term search section incomplete. The documentation should explain the difference between these two important search operations.
Copilot uses AI. Check for mistakes.
|
|
||
| `fulltext` search is Google-stye text search with stop words, stemming. etc. So | ||
| `alloftext: "run woman"` would match "run" as well as "running", etc. For | ||
| example, to find posts that talk about fantastic GraphQL tutorials: |
There was a problem hiding this comment.
The removal of the explanation about fulltext search behavior (lines 286-288) removes important context about how fulltext search differs from term search, including stemming and stop word handling.
| example, to find posts that talk about fantastic GraphQL tutorials: | |
| Fulltext search differs from term search in that it supports stemming and stop word removal. This means that queries using fulltext search will match words with similar roots (e.g., "tutorial" and "tutorials") and ignore common stop words (e.g., "the", "and", "is"). For example, to find posts that talk about fantastic GraphQL tutorials: |
Copilot uses AI. Check for mistakes.
No description provided.