aokolnychyi commented on code in PR #40919:
URL: https://github.com/apache/spark/pull/40919#discussion_r1174639313
##########
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala:
##########
@@ -1818,7 +1818,10 @@ class Analyzer(override val catalogManager:
CatalogManager) extends RuleExecutor
"sqlExpr" -> a.sql,
"cols" -> cols))
}
- resolved
+ resolved match {
Review Comment:
This change is needed as references to nested columns in MERGE are resolved
like `a.nested_col.i AS i` (with added aliases), which breaks our assumption
about what kind of keys we expect. Let me know if anyone can spot any edge
cases when it is not safe.
--
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]
