alamb commented on code in PR #11761:
URL: https://github.com/apache/datafusion/pull/11761#discussion_r1704540401


##########
datafusion/expr/src/expressions/column.rs:
##########


Review Comment:
   THank you all for this -- yes getting the physical-expr and `Expr` types 
untangled will be a great step. I agree with @jayzhan211  that 
https://github.com/apache/datafusion/issues/11359 might be good to look at



##########
datafusion/expr/src/expressions/column.rs:
##########


Review Comment:
   THank you all for this -- yes getting the physical-expr and `Expr` types 
untangled will be a great step. I agree with @jayzhan211  that 
https://github.com/apache/datafusion/issues/11359 might be good to look at first



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

Reply via email to