mingmwang commented on pull request #2024: URL: https://github.com/apache/arrow-datafusion/pull/2024#issuecomment-1075932478
Sorry for the late response. Looks like this PR is a little problematic and it has conflicts with the multiple tenancy SessionContexts. The major problem is in the Executor side, Executor side also need to run those methods to parse the Expr protobuf to physical plan, but Executor side should not have SessionContexts, the optimizers, planner and physical planner does't make sense in Executor side. And for pluggable UDF/UDAF, I think we should have global level UDF/UDAF and session level UDF/UDAF. For example all the build-in UDF/UDAF should be globally available. Different users can also upload/add his own functions to his session. -- 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]
