Eric5553 opened a new pull request #27509: [SPARK-30764][SQL] Improve the 
readability of EXPLAIN FORMATTED style
URL: https://github.com/apache/spark/pull/27509
 
 
   ### What changes were proposed in this pull request?
   The style of **EXPLAIN FORMATTED** output needs to be improved. We’ve 
already got some observations/ideas in
   https://github.com/apache/spark/pull/27368#discussion_r376694496. 
   
   Observations/Ideas:
   1.Using comma as the separator is not clear, especially commas are used 
inside the expressions too.
   2.Show the column counts first? For example, `Results [4]: …`
   3.Currently the attribute names are automatically generated, this need to 
refined.
   4.Add arguments field in common implementations as **EXPLAIN EXTENDED** did 
by calling `argString` in `TreeNode.simpleString`. This will eliminate most 
existing minor differences between 
   **EXPLAIN EXTENDED** and **EXPLAIN FORMATTED**.
   
   This PR is currently addressing comments 2 & 4, and open for more 
discussions on improving readability.
   
   
   ### Why are the changes needed?
   The readability of EXPLAIN FORMATTED need to be improved, which will help 
user better understand the query plan.
   
   
   ### Does this PR introduce any user-facing change?
   Yes, EXPLAIN FORMATTED output style changed.
   
   
   ### How was this patch tested?
   Update expect results of test cases in explain.sql 
   

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to