Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Create built-in function error is hidden from users #1202

Closed
1 of 2 tasks
jiashenC opened this issue Sep 23, 2023 · 0 comments · Fixed by #1204
Closed
1 of 2 tasks

Create built-in function error is hidden from users #1202

jiashenC opened this issue Sep 23, 2023 · 0 comments · Fixed by #1204
Assignees
@jiashenC
Labels
User Experience
Milestone
v0.3.7

Comments

@jiashenC
Copy link
Member

Search before asking

  • I have searched the EvaDB issues and found no similar feature requests.

Description

The executor exception is not exposed. Users have no idea if the built-in functions are not correctly added.

evadb/evadb/functions/function_bootstrap_queries.py

Line 261 in 27e2635

db, query, do_not_print_exceptions=True, do_not_raise_exceptions=True

Use case

No response

Are you willing to submit a PR?

  • Yes I'd like to help by submitting a PR!
@jiashenC jiashenC added the Feature Request ✨ New feature or request label Sep 23, 2023
@jiashenC jiashenC added this to the v0.3.7 milestone Sep 23, 2023
@xzdandy xzdandy added User Experience and removed Feature Request ✨ New feature or request labels Sep 24, 2023
@jiashenC jiashenC linked a pull request Sep 24, 2023 that will close this issue
feat: print error msg if creating built-in functions failed #1204
Merged
@jiashenC jiashenC self-assigned this Sep 24, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@jiashenC jiashenC

Labels
User Experience
Projects
EVA Public Roadmap ⚡🚀
Archived in project
Milestone
v0.3.7
Development

Successfully merging a pull request may close this issue.

feat: print error msg if creating built-in functions failed georgia-tech-db/evadb
2 participants
@xzdandy @jiashenC