cloud-fan commented on a change in pull request #28490: URL: https://github.com/apache/spark/pull/28490#discussion_r470522985
########## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala ########## @@ -1479,6 +1479,33 @@ class Analyzer( // Skip the having clause here, this will be handled in ResolveAggregateFunctions. case h: UnresolvedHaving => h + case agg @ (_: Aggregate | _: GroupingSets) => + val resolved = agg.mapExpressions(resolveExpressionTopDown(_, agg)) + val hasStructField = resolved.expressions.exists { + _.collectFirst { case gsf: GetStructField => gsf }.isDefined + } + if (hasStructField) { + // For struct field, it will be resolve as Alias(GetStructField, name), + // In Aggregate/GroupingSets this behavior will cause same struct field + // in aggExprs/groupExprs/selectedGroupByExprs will be resolved divided + // with different ExprId of Alias and replace failed when construct + // Aggregate in ResolveGroupingAnalytics, so we resolve duplicated struct Review comment: ``` ... will cause the same struct fields in aggExprs/groupExprs/selectedGroupByExprs be treated as different ones due to different ExprIds in Alias, and stops us finding the grouping expressions in aggExprs. Here we resolve ... ``` ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org