johanl-db opened a new pull request, #55239: URL: https://github.com/apache/spark/pull/55239
### What changes were proposed in this pull request? Follow up from https://github.com/apache/spark/pull/54891 in particular [this comment](https://github.com/apache/spark/pull/54891/changes#r3022286893). Adds more tests covering schema evolution in MERGE INTO when the assignment contains an alias. This change also undoes the fix from https://github.com/apache/spark/pull/54891. On closer inspection, this isn't needed in Spark as Spark already removes trivial aliases on nested field accessors in MERGE in [resolveExprInAssignment](https://github.com/apache/spark/blob/b490770d049acfcffa409e16917e92aa3db41f3d/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/ColumnResolutionHelper.scala#L489), which is what the change aimed to allow. Delta doesn't strip aliases during resolution in that way and will require special handling, but that's up to Delta to implement in its custom resolution logic to replicate what Spark already does ### How was this patch tested? Adds tests covering MERGE INTO schema evolution with aliases in assignments -- 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]
