[GitHub] [spark] maropu commented on issue #24759: [SPARK-27395][SQL][WIP] Improve EXPLAIN command

[GitBox <gi...@apache.org>]

maropu commented on issue #24759: [SPARK-27395][SQL][WIP] Improve EXPLAIN command
URL: https://github.com/apache/spark/pull/24759#issuecomment-521857389
 
 
   >> Maybe it's time to introduce a new framework for printing a query plan, which is decoupled from the query plan itself. For example.
   > Wenchen, i think there is value in keeping the string functions in the plan. Its little easier to manage i feel.
   
   sorry for cutting in, but I like the idea; if we had a visiter-pattern-like logic for pringing plans (I think this can be similar to `LogicalPlanVisitor`),  we could easily implement another custom format for printing plans in future. (since the current logic in master is hard-coded, I think its hard to do so though...

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