-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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
Unify tool functions and user defined functions #2101
Comments
Hi @jackgerrits I see this github issue is still open and unassigned. I was notified about this issue while working on #1510 by Linxin Song. Can I contribute to this issue ? |
@jackgerrits is there some advice you can give to @krishnashed regarding this issue? |
We need to come up with a plan and design on how to unify them. One option is to add function execution to the code executor interface, another option is to add a FunctionExecutor interface that follows a similar input format to udfs in code executors so they feel consistent. |
all different with 0.4 |
This one is actually kind of relevant in 0.4 as both features were carried forward |
Currently, tool functions and user defined functions in executors behave fairly differently. We would like to unify the behavior and usage of these so that they are easier to understand and use.
More details to come in this issue.
The text was updated successfully, but these errors were encountered: