adriangb commented on PR #21322:
URL: https://github.com/apache/datafusion/pull/21322#issuecomment-4189695740

   > @martin-g any thoughts on how this should behave w.r.t. field metadata? 
Should we set the metadata to the metadata of the first argument, should we 
merge them or should we preserve only the metadata of the second argument?
   
   I think it makes sense to merge, preferring the target type's metadata? I 
went with that for now.


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