Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-12012][SQL] Backports PR #10004 to branch-1.6 #10250

Closed
wants to merge 1 commit into from

Conversation

liancheng
Copy link
Contributor

No description provided.

…visualizing SQL query plan

This PR adds a `private[sql]` method `metadata` to `SparkPlan`, which can be used to describe detail information about a physical plan during visualization. Specifically, this PR uses this method to provide details of `PhysicalRDD`s translated from a data source relation. For example, a `ParquetRelation` converted from Hive metastore table `default.psrc` is now shown as the following screenshot:

![image](https://cloud.githubusercontent.com/assets/230655/11526657/e10cb7e6-9916-11e5-9afa-f108932ec890.png)

And here is the screenshot for a regular `ParquetRelation` (not converted from Hive metastore table) loaded from a really long path:

![output](https://cloud.githubusercontent.com/assets/230655/11680582/37c66460-9e94-11e5-8f50-842db5309d5a.png)

Author: Cheng Lian <lian@databricks.com>

Closes apache#10004 from liancheng/spark-12012.physical-rdd-metadata.
@liancheng
Copy link
Contributor Author

test this please

@liancheng
Copy link
Contributor Author

Jenkins is shutting down for quite a while (at least two hours).

Nvm, just found out that it's scheduled maintenance.

@liancheng
Copy link
Contributor Author

Actually it doesn't seem to be scheduled. The message "Jenkins is going to shut down" has been there for at least 6 hours.

@SparkQA
Copy link

SparkQA commented Dec 10, 2015

Test build #47521 has finished for PR 10250 at commit 69b3dbd.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@liancheng
Copy link
Contributor Author

retest this please

@SparkQA
Copy link

SparkQA commented Dec 10, 2015

Test build #47523 has finished for PR 10250 at commit 69b3dbd.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@yhuai
Copy link
Contributor

yhuai commented Dec 10, 2015

For backport, it will be great if we still use the original title. I am merging it in and change the title.

asfgit pushed a commit that referenced this pull request Dec 10, 2015
…tadata when visualizing SQL query plan

This PR backports PR #10004 to branch-1.6

It adds a private[sql] method metadata to SparkPlan, which can be used to describe detail information about a physical plan during visualization. Specifically, this PR uses this method to provide details of PhysicalRDDs translated from a data source relation.

Author: Cheng Lian <lian@databricks.com>

Closes #10250 from liancheng/spark-12012.for-1.6.
@yhuai
Copy link
Contributor

yhuai commented Dec 10, 2015

I have merged it. Let's close this PR.

@liancheng liancheng closed this Dec 11, 2015
@liancheng liancheng deleted the spark-12012.for-1.6 branch December 15, 2015 06:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants