You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/03/23 05:44:38 UTC

[GitHub] [arrow-datafusion] mingmwang edited a comment on pull request #2024: Use SessionContext to parse Expr protobuf

mingmwang edited a comment 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 do not make sense in Executor side.
   
   And for the pluggable UDFs/UDAFs, I think we should have global level UDFs/UDAFs and session level UDFs/UDAFs.
   For example all the build-in UDFs/UDAFs should be globally available.  Different users can also upload/add his own functions 
   to his SessionContext.


-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org