Skip to content
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

When appendNumArgs has exclude flag, it should append "(" and num together #143

Closed
xjplke opened this issue Apr 4, 2022 · 0 comments · Fixed by #145
Closed

When appendNumArgs has exclude flag, it should append "(" and num together #143

xjplke opened this issue Apr 4, 2022 · 0 comments · Fixed by #145

Comments

@xjplke
Copy link
Contributor

xjplke commented Apr 4, 2022

return append(args, "(", num)

When appendNumArgs has exclude flag, it should append "(" and num together.

The command line of FT.SEARCH is like this
127.0.0.1:6379> FT.SEARCH myIndex "hello world" LIMIT 0 2 RETURN 1 title FILTER age 29 (31

  1. (integer) 1
  2. "doc1"
    1. "title"
    2. "Hello world"

127.0.0.1:6379> FT.SEARCH myIndex "hello world" LIMIT 0 2 RETURN 1 title FILTER age 29 ( 31
(error) Unknown argument 31 at position 11 for

xjplke pushed a commit to xjplke/redisearch-go that referenced this issue Apr 4, 2022
Avital-Fine added a commit that referenced this issue Apr 27, 2022
Co-authored-by: shaojunwu <xjplke@163.com>
Co-authored-by: Avital Fine <98389525+Avital-Fine@users.noreply.github.com>
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 a pull request may close this issue.

1 participant