viirya commented on a change in pull request #24464: [SPARK-27439][SQL] Use existing queryexecution in explaining Dataset URL: https://github.com/apache/spark/pull/24464#discussion_r280050973
########## File path: sql/core/src/main/scala/org/apache/spark/sql/execution/command/commands.scala ########## @@ -134,13 +134,13 @@ case class DataWritingCommandExec(cmd: DataWritingCommand, child: SparkPlan) * EXPLAIN (EXTENDED | CODEGEN) SELECT * FROM ... * }}} * - * @param logicalPlan plan to explain + * @param queryExecution the query execution object of the plan to explain * @param extended whether to do extended explain or not * @param codegen whether to output generated code from whole-stage codegen or not * @param cost whether to show cost information for operators. */ case class ExplainCommand( - logicalPlan: LogicalPlan, + queryExecution: QueryExecution, Review comment: @mgaido91 `ExplainCommand` now accepts only `QueryExecution`. ---------------------------------------------------------------- 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]
