Skip to content

Conversation

@filipecosta90
Copy link
Collaborator

added support for FT.SPELLCHECK.

the optional parameters:

  • inclusion named dictionary
  • exclusion named dictionary
  • distance
    are all supported with the addition of SpellCheckOptions struct.

Regarding the command response handling, the method SpellCheck with signature:

func (i *Client) SpellCheck(q *Query, s *SpellCheckOptions) (suggs []MisspelledTerm, total int, err error)

returns:

  • suggestions for misspelled terms in the format of a slice of MisspelledTerm (contains the misspelled term and a sortable list of suggestions)
  • the total number of results
  • or an error if something went wrong

Copy link

@MeirShpilraien MeirShpilraien left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, small comments

Copy link

@MeirShpilraien MeirShpilraien left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo, after you fix you can merge :)

MeirShpilraien
MeirShpilraien previously approved these changes Sep 24, 2019
MeirShpilraien
MeirShpilraien previously approved these changes Oct 3, 2019
@filipecosta90 filipecosta90 merged commit 18efbc5 into RediSearch:master Oct 28, 2019
@filipecosta90 filipecosta90 deleted the spellcheck branch April 24, 2020 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants