maropu commented on issue #24759: [SPARK-27395][SQL][WIP] Improve EXPLAIN command URL: https://github.com/apache/spark/pull/24759#issuecomment-514025812 oh I see, since I've never used db2, I didn't notice that (this output is based on the db2 one). Thanks! I think the current format is basically ok to me. My suggestion is just that we could make the format more better by referring EXPLAIN output in the other systems. e.g., in the current format in this pr, only a plan structure is written in the top and all the plan contents are written in the bottom. As another option, we might be able to leave some basic contents (e.g., filter conditions?) in the top as following; ``` ... +- Filter (isnotnull(val#29) AND (val#29 > 0)) (11) +- ... ``` The postgresql format does so (only basic plan contents are printed in explain). There is a room to discuss which content is basic or not 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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
