mgaido91 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_r280056701
##########
File path: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala
##########
@@ -494,11 +494,15 @@ class Dataset[T] private[sql](
/**
* Prints the plans (logical and physical) to the console for debugging
purposes.
*
+ * Note that temporary views are already resolved when creating `Dataset`.
So if
+ * temporary views are changed after that, the output of this command shows
the plans
+ * before such changes.
Review comment:
I am not sure whether this is the right place for this comment. I mean, I
think that putting it here, it is not clear whether the output of the `explain`
shows the plan with "old" views, but the dataset is executed with the "new"
ones or both use the "old", as it is.
----------------------------------------------------------------
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]
