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.

Sign up for GitHub

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

Jump to bottom

Python client does not allow to create Text field with noindex #34

Closed
nhapq opened this issue Mar 28, 2019 · 0 comments · Fixed by #118
Closed

Python client does not allow to create Text field with noindex #34

nhapq opened this issue Mar 28, 2019 · 0 comments · Fixed by #118
Assignees
@leibale
Labels
enhancement

Comments

@nhapq
Copy link

nhapq commented Mar 28, 2019

While we can create a noindex nosterm textfield by directly executing on redis-cli:

127.0.0.1:6379> FT.CREATE testidx SCHEMA name TEXT NOSTEM NOINDEX
OK
But Python Client does not allow us to do that, it raises exception:
Non-Sortable non-Indexable fields are ignored

https://github.com/RediSearch/redisearch-py/blob/master/redisearch/client.py
@gkorland gkorland assigned leibale and unassigned mnunberg Apr 11, 2021
leibale added a commit that referenced this issue Apr 13, 2021
@leibale
fix #34 #116 - move sortable and no_index to the field field abstract…
9fb5960 
… class
@leibale leibale mentioned this issue Apr 13, 2021
Merged
gkorland pushed a commit that referenced this issue May 6, 2021
@leibale
move sortable and no_index to the field field abstract… (#118)
5aa42a6 
* fix #34 #116 - move sortable and no_index to the field field abstract class
* add tests for no_index and sortable flags
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@leibale leibale

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

move sortable and no_index to the field field abstract… RediSearch/redisearch-py
4 participants
@mnunberg @gkorland @nhapq @leibale