goldmedal commented on issue #6518: URL: https://github.com/apache/datafusion/issues/6518#issuecomment-2671902644
> I think the biggest piece of work needed a if we ported https://github.com/goldmedal/datafusion-llm-function/pull/1 to the core would be tests and support for other plan nodes (https://github.com/goldmedal/datafusion-llm-function/pull/1 only does ProjectionExec, but supporting others I think is more mechanical) I'd like to port the implementation to the core, possibly by first implementing the trait for async UDFs and the corresponding planning behavior. As for the LLM functions, I think they would be better suited for a separate datafusion-contrib repo. I'm also considering implementing this at the logical plan level. Some thoughts here: https://github.com/goldmedal/datafusion-llm-function/issues/6. Feel free to share your opinion. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
