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/07/24 09:16:15 UTC

[GitHub] [spark] peter-toth edited a comment on pull request #29210: [WIP][SPARK-24497][SQL] Support recursive SQL query

peter-toth edited a comment 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 don'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 a 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:
users@infra.apache.org



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