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

Missing Similarity Function #1189

Open
1 of 2 tasks
jarulraj opened this issue Sep 22, 2023 · 8 comments
Open
1 of 2 tasks

Missing Similarity Function #1189

jarulraj opened this issue Sep 22, 2023 · 8 comments
Assignees
Labels

Comments

@jarulraj
Copy link
Member

jarulraj commented Sep 22, 2023

Search before asking

  • I have searched the EvaDB issues and found no similar bug report.

Bug

Function 'Similarity' does not exist in the catalog. Please create the function using CREATE UDF command.

Need to change it to CREATE FUNCTION. It would be neat if we could suggest a closely named function if there is a typo in the query.

Environment

No response

Are you willing to submit a PR?

  • Yes I'd like to help by submitting a PR!
@jarulraj jarulraj added the Feature Request ✨ New feature or request label Sep 22, 2023
@xzdandy xzdandy added Bug 🐞 EVA is not working as expected User Experience labels Sep 22, 2023
@xzdandy xzdandy added this to the v0.3.7 milestone Sep 22, 2023
@xzdandy xzdandy removed the Bug 🐞 EVA is not working as expected label Sep 22, 2023
@xzdandy xzdandy removed this from the v0.3.7 milestone Sep 30, 2023
@jarulraj
Copy link
Member Author

jarulraj commented Oct 4, 2023

@hershd23 It looks like the fuzzing logic is needed for both table name and function name typos.

@hershd23
Copy link
Contributor

hershd23 commented Oct 4, 2023

Sure @jarulraj

@hershd23
Copy link
Contributor

I ran the similarity checks in the long integration tests and they work fine. I get the fuzzy error message logic part for the functions. Can you let me know if I am not getting something? Similarity tests are working fine so don't understand what the issue is with that

@hershd23
Copy link
Contributor

Also see the Similarity function in bootstrap queries

Similarity_function_query = """CREATE FUNCTION IF NOT EXISTS Similarity

@xzdandy
Copy link
Collaborator

xzdandy commented Oct 17, 2023

Hi @hershd23, I think what Joy meant is the error message at https://github.com/georgia-tech-db/evadb/blob/staging/evadb/binder/statement_binder.py#L287. This has been already fixed.

One improvement is to suggest possible functions, when there is a typo in function name. For reference, we did that for columns at https://github.com/georgia-tech-db/evadb/blob/staging/evadb/binder/statement_binder_context.py#L155

@Omanshb
Copy link

Omanshb commented Oct 18, 2023

I will be taking this issue up with Hersh.

@hershd23
Copy link
Contributor

@xzdandy do you how we can get a list of all function names/identifiers. I think that will make this issue pretty trivial

@xzdandy
Copy link
Collaborator

xzdandy commented Oct 19, 2023

@xzdandy xzdandy moved this from ToDo to In Progress in EVA Public Roadmap ⚡🚀 Oct 19, 2023
@xzdandy xzdandy added this to the v0.3.9 milestone Oct 19, 2023
@xzdandy xzdandy removed this from the v0.3.9 milestone Nov 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: In Progress
Development

No branches or pull requests

4 participants