Re: [PR] [SPARK-52486][SQL] Fix Spark Driver Planning OOM issue due to unworthwhile dpp expression before Execution when enabling AQE [spark]

["zhixingheyi-tian (via GitHub)" <gi...@apache.org>]

zhixingheyi-tian commented on PR #51184:
URL: https://github.com/apache/spark/pull/51184#issuecomment-2983016486

   > Thank you for making a PR, @zhixingheyi-tian . I have two comments.
   > 
   > 1. Please provide a reproducible example to clarify and validate your claim.
   > 2. Please revise the PR title more specifically because the current title is misleading because OOM depends on the situation. Technically, this PR cannot prevent all kind of Driver Planning OOM, right?
   
   @dongjoon-hyun 
   
   Thanks your review.
   A reproducible example can refer to SPARK-52486;
   The PR title has been revised;
   The benefit description is added in the PR description.
   


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