dtenedor commented on PR #48754: URL: https://github.com/apache/spark/pull/48754#issuecomment-2465418016
Looking at this more, the proposed semantics would not be consistent with how aggregation works with pipe operators, where the GROUP BY expressions arrive out of the operator followed by the aggregate functions. I will close this. As an aside, we might want to consider returning errors if any of the aggregate functions contain column references that are not aggregated, since the pipe operator returns the grouping expressions and it is always possible to use a projection (`|> SELECT`) afterwards based on the grouping/aggregate functions. -- 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]
