cloud-fan commented on issue #22957: [SPARK-25951][SQL] Ignore aliases for distributions and orderings URL: https://github.com/apache/spark/pull/22957#issuecomment-459293799 So we have the same understanding of the root cause, but we have different ideas about how to fix it. My suggestion is to fix `ProjectExec.outputPartitioning`(and agg/window as well) directly, instead of introducing `Expression.sameResult` and fixing the problem at where we look at the partitioning/ordering. We may have more places relying on partitioning/ordering in the future, it's better to fix it at where we produce it.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
