jchen5 commented on PR #45133: URL: https://github.com/apache/spark/pull/45133#issuecomment-1965429407
Yeah, in the prior examples it worked because the group-by columns will determine the value of the exists - that was only true when the aggregates did not use any non-group-by columns of the table. When we do have an aggregate using a non-grouped column of the outer table it wouldn't work. This aggregate doesn't require any changes to avoid the invalid aggregation error, right? I think this is another reason to go towards something like wrapping expressions in any_value aggregate when needed. -- 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: reviews-unsubscr...@spark.apache.org 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
