alamb commented on pull request #1881: URL: https://github.com/apache/arrow-datafusion/pull/1881#issuecomment-1057142655
> Yes, the udf plugin is designed for those who use Ballista as a computing engine, but do not want to modify the source code of ballista. ... I believe this is a more friendly way for those who actually use ballista as a computing engine. I agree and this makes sense > In my opinion, people who use datafusion and people who use ballista are different people, and the udf plugin is more suitable for ballista than datafusion. 💯 agree here too > I don't think scalar_functions and aggregate_functions in ExecutionContext need to be modified as these are for those who use datafusion but not ballista. So I think I should modify the code and migrate the plugin mod into the ballista crate instead of staying in datafusion. I think the idea of moving the plugin module into ballista makes a lot of sense to me > Thanks a lot, can you give me more advice on these? Thank you for your clear explination and justification 👍 -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
