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 2019/05/16 14:43:47 UTC

[GitHub] [spark] cloud-fan opened a new pull request #24626: [SPARK-27747][SQL] add a logical plan link in the physical plan

cloud-fan opened a new pull request #24626: [SPARK-27747][SQL] add a logical plan link in the physical plan
URL: https://github.com/apache/spark/pull/24626
 
 
   ## What changes were proposed in this pull request?
   
   It's pretty useful if we can convert a physical plan back to a logical plan, e.g., in https://github.com/apache/spark/pull/24389
   
   This PR introduces a new feature to `TreeNode`, which allows `TreeNode` to carry some extra information via a mutable map, and keep the information when it's copied.
   
   The planner leverages this feature to put the logical plan into the physical plan.
   
   ## How was this patch tested?
   
   a test suite that runs all TPCDS queries and checks that some common physical plans contain the corresponding logical plans.

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


With regards,
Apache Git Services

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