alamb commented on issue #8045:
URL: 
https://github.com/apache/arrow-datafusion/issues/8045#issuecomment-1793147864

   > Would like to add that supporting serialization of user-defined functions 
would be quite nice. 
   
   I don't understand this question @thinkharderdev 🤔 
   
   The current approach to serialize a `ScalarUDF` as I understand it, is to 
send over the function name along with the arguments
   
   
https://github.com/apache/arrow-datafusion/blob/c2e768052c43e4bab6705ee76befc19de383c2cb/datafusion/proto/proto/datafusion.proto#L686-L689
   
   And then the physical plan deserialization looks that function up by name 
https://docs.rs/datafusion-proto/latest/src/datafusion_proto/physical_plan/from_proto.rs.html#318-333
   
   The only difference compared to `BuiltInScalarFunction` is that 
`BuiltInScalarFunction` uses a number (in an enum) which might be slightly 
faster


-- 
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]

Reply via email to