-
Notifications
You must be signed in to change notification settings - Fork 508
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
[DOC] updating legacy index API sorting.md #3930
base: master
Are you sure you want to change the base?
Conversation
updating the code example from the legacy FT.ADD to simple HSET, and fix some bugs
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.
There are probably more places in our docs where we use ft.add
and other deprecated commands, consider changing these in this PR as well
|
||
# Searching while sorting | ||
|
||
# Searching by last name and sorting by first name | ||
> FT.SEARCH users "@last_name:jones" SORTBY first_name DESC | ||
|
||
# Searching by both first and last name, and sorting by age | ||
> FT.SEARCH users "alice jones" SORTBY age ASC | ||
> FT.SEARCH users "jones" SORTBY age ASC |
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.
Why the change here?
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.
compound names didn't work int his example
Changes LGTM. |
This pull request is stale because it has been open for 60 days with no activity. |
The changes in this PR will be merged to the new docs site. |
updating the code example from the legacy FT.ADD to simple HSET, and fix some bugs
Describe the changes in the pull request
A clear and concise description of what the PR is solving, including:
Which issues this PR fixes
Main objects this PR modified
Mark if applicable