jayzhan211 commented on PR #14094:
URL: https://github.com/apache/datafusion/pull/14094#issuecomment-2585752891
```rust
#[derive(Debug)]
pub enum ReturnTypeArgs<'a> {
/// information known at logical planning time
/// Note you can get get type and nullability for each arg
// using the specified ExprSchema
Planning {
pub args: &'a[Expr],
pub schema: &'a dyn ExprSchema
},
/// Information known during Execution
Execution {
/// The data types of the arguments to the function
pub arg_types: &'a [DataType],
pub arg_nullability: [bool],
}
}
```
One good thing in this PR is that we don't need `Expr` anymore, we compute
data type and nullable in datafusion core and they are not "public" for
customization.
Do we really need `Planning`?
--
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-unsubscr...@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org
