manticoresearch icon indicating copy to clipboard operation
manticoresearch copied to clipboard

exact form modified vs expanded keywords

Open sanikolaev opened this issue 4 years ago • 10 comments

Currently exact form modified = doesn't prevent expanding keywords, e.g.:

mysql> drop table if exists t;create table t(f text) min_infix_len='1' expand_keywords='1'; insert into t values(1,'abc'),(2,'bc');  
mysql> select * from t where match('=bc');  
 ------ ------   
| id   | f    |  
 ------ ------   
|    2 | bc   |  
|    1 | abc  |  
 ------ ------   
2 rows in set (0.00 sec)  
mysql> show meta;  
 --------------- -------   
| Variable_name | Value |  
 --------------- -------   
| total         | 2     |  
| total_found   | 2     |  
| time          | 0.002 |  
| keyword[0]    | *bc*  |  
| docs[0]       | 2     |  
| hits[0]       | 2     |  
| keyword[1]    | bc    |  
| docs[1]       | 1     |  
| hits[1]       | 1     |  
 --------------- -------   
9 rows in set (0.00 sec)  

It would be good to fix it.

sanikolaev avatar Dec 10 '20 09:12 sanikolaev

➤ Stan commented:

also need to check wildcard modifiers, ie term? not get replaced by *term*

githubmanticore avatar Dec 10 '20 14:12 githubmanticore

@sanikolaev @githubmanticore

Hello! Any news?

glukkkk avatar Jan 11 '21 08:01 glukkkk

No news. This task is not in our nearest plans, but we'll be happy to review and accept a pull request.

sanikolaev avatar Jan 13 '21 04:01 sanikolaev

@sanikolaev Maybe there is some workaround for this?

glukkkk avatar Jan 13 '21 04:01 glukkkk

expand_keywords=0 ?

sanikolaev avatar Jan 13 '21 05:01 sanikolaev

Or expand_keywords=exact

sanikolaev avatar Jan 13 '21 05:01 sanikolaev

@sanikolaev In fact we need to keep expand_keywords = 1, but have the ability to search by exact phrase.

glukkkk avatar Jan 13 '21 05:01 glukkkk

Can't think of another workaround.

sanikolaev avatar Jan 13 '21 05:01 sanikolaev

@sanikolaev Hello! Half a year has passed :) Maybe you have any good news for me?

glukkkk avatar Jun 21 '21 09:06 glukkkk

Hello @glukkkk

Sure, the good news are that:

All these tasks were of a higher priority for us since we believe it's more important for the community at the moment or someone wanted to sponsor development of some of these features, which allowed us to sustain the development in general. Unfortunately, this issue is of a lower priority and will be so until we are done with more important things or until someone makes a merge request. Thank you for your understanding.

sanikolaev avatar Jun 21 '21 09:06 sanikolaev