Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/16027#discussion_r89716395
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/CollapseWindowSuite.scala
---
@@ -46,12 +46,15 @@ class
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/16027#discussion_r89716308
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -545,7 +545,7 @@ object CollapseRepartition
GitHub user hvanhovell opened a pull request:
https://github.com/apache/spark/pull/16027
Make sure CollapseWindow returns the attributes in the same order.
## What changes were proposed in this pull request?
The `CollapseWindow` optimizer rule changes the order of output
