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/31 16:45:13 UTC

[GitHub] [spark] dilipbiswal edited a comment on issue #24759: [SPARK-27395][SQL][WIP] Improve EXPLAIN command

dilipbiswal edited a comment on issue #24759: [SPARK-27395][SQL][WIP] Improve EXPLAIN command
URL: https://github.com/apache/spark/pull/24759#issuecomment-497779919
 
 
   @ekoifman Thanks for your comment. So currently, `verboseString` has not been implemented for all the operators. What we have is very basic information like a default implementation. We will need to implement `verboseString` for each operator that will print the necessary information for each op.
   
   > it may be useful to support "EXPLAIN FORMATTED" or something like this for new style, and "EXPLAIN" for old - may break tests otherwise.
   
   Yeah. Its definitely an option. Lets get some more feedback and we can decide on it.

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