alamb commented on code in PR #20676:
URL: https://github.com/apache/datafusion/pull/20676#discussion_r2884001897
##########
datafusion/expr/src/planner.rs:
##########
@@ -440,4 +440,17 @@ pub trait TypePlanner: Debug + Send + Sync {
) -> Result<Option<DataType>> {
Ok(None)
}
+
+ /// Plan SQL [`sqlparser::ast::DataType`] to DataFusion [`FieldRef`]
+ ///
+ /// Returns None if not possible. The default implementation falls back
+ /// on plan_type and wraps it in a nullable field reference.
+ fn plan_type_field(
Review Comment:
Maybe we can help document how this is different from plan_type (or should
we be deprecating plan_type in favor of `plan_type_field`?
--
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]
