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

Misleading error "unknown option 'fuzzy'" #395

Closed
5 tasks done
sanikolaev opened this issue Nov 13, 2024 · 6 comments
Closed
5 tasks done

Misleading error "unknown option 'fuzzy'" #395

sanikolaev opened this issue Nov 13, 2024 · 6 comments
Assignees
Labels
bug Something isn't working

Comments

@sanikolaev
Copy link
Collaborator

sanikolaev commented Nov 13, 2024

Bug Description:

Here:

mysql> drop table if exists t; create table t(a int); select * from t option fuzzy=1;
--------------
drop table if exists t
--------------

Query OK, 0 rows affected (0.00 sec)

--------------
create table t(a int)
--------------

Query OK, 0 rows affected (0.00 sec)

--------------
select * from t option fuzzy=1
--------------

ERROR 1064 (42000): unknown option 'fuzzy'

the error should be different, I suggest:

ERROR 1064 (42000): The 'fuzzy' option requires a full-text query

Manticore Search Version:

Manticore 6.3.7 c91fb60b1@24111216 dev (columnar 2.3.1 30ad2d6@24100914) (secondary 2.3.1 30ad2d6@24100914) (knn 2.3.1 30ad2d6@24100914)

Operating System Version:

Ubuntu Jammy (dev2)

Have you tried the latest development version?

Yes

Internal Checklist:

To be completed by the assignee. Check off tasks that have been completed or are not applicable.

  • Implementation completed
  • Tests developed
  • Documentation updated
  • Documentation reviewed
  • Changelog updated
@donhardman
Copy link
Contributor

Implemented: #397

@donhardman
Copy link
Contributor

Let's add tests that MRE is fixed in this pull: manticoresoftware/manticoresearch#2756

@PavelShilin89
Copy link

@donhardman Testing performed in PR - manticoresoftware/manticoresearch#2756
I ask for a test review and approval for the merger.

@donhardman
Copy link
Contributor

Approved

@PavelShilin89
Copy link

@sanikolaev do you need to tick the changelog?

@sanikolaev
Copy link
Collaborator Author

Done, but no record in the changelog is required since the main functionality hasn't been released yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants