alamb commented on issue #17330:
URL: https://github.com/apache/datafusion/issues/17330#issuecomment-3615853726

   @kosiew  -- I recently ran into `CastColumnExpr` in the context of 
https://github.com/apache/datafusion/pull/18998
   
   It doesn't seem to be connected to anything (I removed it and all tests pass 
https://github.com/apache/datafusion/pull/19100)
   
   I also took a shot at integrating the logic for struct casting into 
`CaseExpr` and it seemed to go reasonably well
   - https://github.com/apache/datafusion/pull/19097
   
   So I was wondering if you had any thoughts on how you would like to proceed 
   
   I think integrating the logic into CaseExpr is the most elegant long term 
solution, but since you added `CastColumnExpr` I wanted to double check 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