alamb commented on PR #24094:
URL: https://github.com/apache/datafusion/pull/24094#issuecomment-5193439470

   > Thanks for working on this. The plan-visible CoerceSchemaExec approach 
looks good, and I like that it removes the schema restamping from 
UnionExec::execute() and InterleaveExec::execute() while keeping the coercion 
explicit in the plan.
   
   I wonder if instead of using an entirely new exec, we could use the existing 
ProjectionExec operator with Cast exprs? In theory it should already be capable 
of doing the schema tranformation (assuming the physical CastExpr can have a 
field -- not just a DataType)
   
   Thank you for follow up on this @dariocurr and @kosiew 


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