paleolimbot commented on PR #23169: URL: https://github.com/apache/datafusion/pull/23169#issuecomment-5497011224
> think these (this PR) are the right long term semantics we want or we think we should backport something to 55 and this is our best option then I'm 👍🏻 on it Whether the target field obliterates incoming metadata or merges it doesn't matter much (except for the extension name, which this PR handles); passthrough is safer because DataFusion has done it for a long time, and can be changed by an optimizer rule if a database needs it. I would love it if this were merged and backported while we're all here looking at it, and any improvements / breaking changes to code or behaviour can be a separate discussion. -- 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]
