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


   This is the static version of https://github.com/apache/spark/pull/23531 
which means this PR doesn't do adaptive replanning in each recursive iteration, 
but the advantage is that the implementation comes with simplified code. Common 
relational DB implementations that doesn't do replanning either.
   
   @maropu this is very close to what you suggested before, if you have some 
time please review.
   
   @maryannxue I think we can add adaptive support in follow-up PR if needed.
   
   cc @cloud-fan @viirya @gatorsmile 


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

Reply via email to