bersprockets commented on PR #37825: URL: https://github.com/apache/spark/pull/37825#issuecomment-1243010034
> #34953 take a look this pr? If I understand that PR correctly (and I may not), it is related but orthogonal. One PR doesn't preclude the other. The PR you reference is making the `Expand` projections narrower (by reusing slots that have the same data type) and including the `then` expression of if-like expressions in the grouping of distinct aggregates -- thus potentially reducing the number of projections. I think that PR would still apply to the code even after my propose changes. -- 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]
