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 2020/06/21 09:19:24 UTC

[GitHub] [spark] peter-toth commented on pull request #28881: [SPARK-32041][SQL] Fix Exchange reuse issues when subqueries are involved

peter-toth commented on pull request #28881:
URL: https://github.com/apache/spark/pull/28881#issuecomment-647102573


   @prakharjain09 , it seems we both opened PRs (https://github.com/apache/spark/pull/28885 is mine) to fix the issue with exchange and subquery reuse. It looks like we came to the same conclusion that the separate reuse rules needs to be unified. My PR does a bit more that that and actually does the combined reuse in a bit different way than yours. I also see that you opened the ticket SPARK-32041 for the issue. If you don't mind I would add that ticket to my PR as well.


----------------------------------------------------------------
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.

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