peter-toth commented on pull request #28885: URL: https://github.com/apache/spark/pull/28885#issuecomment-763437539
I've updated this PR with minor changes due to AQE is now enabled by default. Without this PR invalid reuse nodes are in many TPCDS queries causing performance degradation: q14a, q14b, q23a, q23b, q47, q57, q5a-v2.7, q14-v2.7, q14a-v2.7, q22a-v2.7, q36a-v2.7, q47-v2.7, q51a-v2.7, q57-v2.7, q67a-v2.7, q70a-v2.7, q77a-v2.7, q80a-v2.7, q86a-v2.7 https://github.com/apache/spark/pull/31243 updates PlanStabilitySuite tests to cover partitioned tables (DPP kicks in) and the issue appears as `ReusedExchange [Reuses operator id: unknown]` nodes in `explain.txt`s of that PR. @cloud-fan @viirya @wangyum could you please review this PR? ---------------------------------------------------------------- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
