fix: use proper indexes for full text search #4787
Conversation
|
|
||
| // for use lower for like and ilike | ||
| if op == v3.FilterOperatorLike || op == v3.FilterOperatorNotLike { | ||
| columnName = fmt.Sprintf("lower(%s)", columnName) |
There was a problem hiding this comment.
Can you share some examples with and without lower on mixed case data? The way I imagined this would work is that we need to make the index lower and convert the search value to lower but do not use the lower function. Just to double check we will not run into this kind of problem https://stackoverflow.com/questions/62629579/mysql-why-using-where-on-date-column-does-not-use-index
There was a problem hiding this comment.
This is tested , using lower(body) in the where clause uses the corresponding lower(body) index
#4259 (comment)
There was a problem hiding this comment.
Can I play with this somewhere?
There was a problem hiding this comment.
please check slack
|
|
||
| // for use lower for like and ilike | ||
| if op == v3.FilterOperatorLike || op == v3.FilterOperatorNotLike { | ||
| columnName = fmt.Sprintf("lower(%s)", columnName) |
There was a problem hiding this comment.
Can I play with this somewhere?
Fixes #4259
changes
containswill also uselower(attribute)now but there is no change for them.