search

RediSearch / redisearch-go

Go client for RediSearch
https://redisearch.io
BSD 3-Clause "New" or "Revised" License
293 stars 65 forks source link

Fixing index creation: CASESENSITIVE preceeds SORTABLE #173

Closed Avital-Fine closed 2 years ago

Avital-Fine commented 2 years ago

CASESENSITIVE label should be declared before SORTABLE:

127.0.0.1:6379>  FT.CREATE ft-info-fields-test ON HASH PREFIX 1 ft-info-fields-test: SCHEMA t as tag TAG SEPARATOR "." SORTABLE CASESENSITIVE
(error) Field `CASESENSITIVE` does not have a type
127.0.0.1:6379>  FT.CREATE ft-info-fields-test ON HASH PREFIX 1 ft-info-fields-test: SCHEMA t as tag TAG SEPARATOR "." CASESENSITIVE SORTABLE
OK
codecov[bot] commented 2 years ago

Codecov Report

Merging #173 (705ecf1) into master (c049342) will increase coverage by 0.41%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #173      +/-   ##
==========================================
+ Coverage   82.46%   82.87%   +0.41%     
==========================================
  Files          13       13              
  Lines        1460     1460              
==========================================
+ Hits         1204     1210       +6     
+ Misses        199      196       -3     
+ Partials       57       54       -3
Impacted Files Coverage Δ
redisearch/schema.go 92.10% <100.00%> (ø)
redisearch/client.go 82.68% <0.00%> (+1.29%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Avital-Fine commented 2 years ago

@chayim

sonarcloud[bot] commented 2 years ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication