You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2022/08/03 09:06:16 UTC

[GitHub] [spark] peter-toth commented on pull request #35334: [SPARK-38034][SQL] Optimize TransposeWindow rule

peter-toth commented on PR #35334:
URL: https://github.com/apache/spark/pull/35334#issuecomment-1203684886

   @cloud-fan, @hvanhovell, I think this is a pretty good optimization.
   The old `compatiblePartitions()` was very compute intensive if `Window` nodes have a wide `partitionSpec`s.
   
   We have a customer who upgraded from Spark 2 to Spark 3 and due to the new `TransposeWindow` rule in Spark 3  suffered severe performance degradation...
   
   The PR probably needs a new ticket though.


-- 
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-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org