gstvg commented on PR #18921: URL: https://github.com/apache/datafusion/pull/18921#issuecomment-3674167149
Thanks @linhr I think a new `LambdaUDF` is reasonable, and I'm not strongly inclined towards using `ScalarUDF` either >Expr::Lambda and Expr::LambdaColumn are only fragments whose data types etc. are not well-defined which may be challenging to work with in other parts of the library (e.g. ExprSchemable). Yeah, this is the major challenge in this PR. Currently, `Expr::Lambda` always return `DataType::Null` and `Expr::LambdaVariable` embeds a `FieldRef` which is used to implement the `ExprSchemable` and don't even look at the `DFSchema` >when the function is actually "invoked" with Arrow arrays, the lambda parameter should have been resolved and removed from the argument list. Could you expand this further? It is like `LambdaUDF` having a `resolve_lambdas` method which result is passed to the `invoke_with_args` method? -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
