[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-12-17 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/7774#discussion_r47942980 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/ui/SQLListenerSuite.scala --- @@ -0,0 +1,347 @@ +/* + * Licensed to the Apache Software Foundat

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-12-17 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/7774#discussion_r47942483 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/ui/SQLListenerSuite.scala --- @@ -0,0 +1,347 @@ +/* + * Licensed to the Apache Software Founda

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-10-09 Thread v-gerasimov
Github user v-gerasimov commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-147038637 @davies Yes, it sounds good. Thanks for everyone! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-10-09 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-146947985 @v-gerasimov We have introduced a new API (newSession(), #8909) to improve the support for multiple sessions, the SQLTab will be shared by multiple sessions (#9048), does

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-10-08 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-146630775 @v-gerasimov we will find a solution for this in 1.6. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If you

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-10-08 Thread v-gerasimov
Github user v-gerasimov commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-146589821 @zsxwing No, I can't. I have one spark context for different applications which registers own temp tables with same names. What if we hide SQL tabs in dropdown menu?

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-10-08 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-146581482 > @zsxwing Why did you do that? It looks really dirty when I try to use Spark UI. Maybe Spark UI header isn't a good place for SQL tabs. What do you think? It's

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-10-08 Thread v-gerasimov
Github user v-gerasimov commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-146564907 @zsxwing Why did you do that? It looks really dirty when I try to use Spark UI. --- If your project is set up for it, you can reply to this email and have your repl

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-10-08 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-146559430 > @zsxwing Yes, I did. Do you think it is wrong? That's expected. Every SQLContext has a separate tab to track it's own queries. --- If your project is set up

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-10-08 Thread v-gerasimov
Github user v-gerasimov commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-146539385 @zsxwing Yes, I did. Do you think it is wrong? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If you

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-10-08 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-146535686 > Hi, @zsxwing! > > It seems like something wrong when you try to add SQLTab to SparkUI.tabs, doesn't it?(https://github.com/dennishuo/spark/blob/c791828ccf2873

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-10-08 Thread v-gerasimov
Github user v-gerasimov commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-146475733 Hi, @zsxwing! It seems like something wrong when you try to add SQLTab to SparkUI.tabs, doesn't it?(https://github.com/dennishuo/spark/blob/c791828ccf2873c0

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-08 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-129081760 I just realized I missed pasting the first two lines of the error ``` error: error while loading SQLListener, Missing dependency 'bad symbolic reference. A

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-08 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-129021145 @zsxwing @rxin I just ran into a problem with the annotation thing that was discussed earlier in this PR. My master branch is clean and is at https://github.com/ap

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-05 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-128020291 Review status: 4 of 24 files reviewed at latest revision, 72 unresolved discussions, all commit checks successful. --- **[sql/core/s

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-05 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/7774 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enab

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-05 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127919576 I've merged this. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this featur

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127919017 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-05 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127918770 [Test build #39828 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39828/console) for PR 7774 at commit [`5a2bc99`](https://github.

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-05 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127912066 [Test build #1349 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1349/console) for PR 7774 at commit [`57d4cd2`](https://github

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-05 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127906718 [Test build #1348 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1348/console) for PR 7774 at commit [`57d4cd2`](https://github

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-05 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127899454 Verified locally that `org.seleniumhq.selenium:selenium-java:test` will introduce some incompatible dependency (may be a different version of guava?) and make the tests

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127886593 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127886731 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127887044 [Test build #39828 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39828/consoleFull) for PR 7774 at commit [`5a2bc99`](https://gith

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127884863 [Test build #1349 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1349/consoleFull) for PR 7774 at commit [`57d4cd2`](https://git

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127880431 [Test build #1348 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1348/consoleFull) for PR 7774 at commit [`57d4cd2`](https://git

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127832514 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127832475 [Test build #220 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SlowSparkPullRequestBuilder/220/console) for PR 7774 at commit [`57d4cd2`](https://github.

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127830429 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127830244 [Test build #39794 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39794/console) for PR 7774 at commit [`57d4cd2`](https://github.

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127809375 [Test build #39794 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39794/consoleFull) for PR 7774 at commit [`57d4cd2`](https://gith

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127809001 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127808966 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127808816 [Test build #220 has started](https://amplab.cs.berkeley.edu/jenkins/job/SlowSparkPullRequestBuilder/220/consoleFull) for PR 7774 at commit [`57d4cd2`](https://gith

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127808533 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127808558 Trying to run `org.apache.spark.sql.hive.client.VersionsSuite` locally. It needs to download all versions of Hive and is very slow in my machine... --- If your project

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127808492 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127808087 retest this please. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this fe

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127807213 [Test build #1339 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1339/console) for PR 7774 at commit [`57d4cd2`](https://github

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127780194 [Test build #1339 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1339/consoleFull) for PR 7774 at commit [`57d4cd2`](https://git

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127727439 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127727392 [Test build #39708 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39708/console) for PR 7774 at commit [`57d4cd2`](https://github.

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127725070 [Test build #206 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SlowSparkPullRequestBuilder/206/console) for PR 7774 at commit [`57d4cd2`](https://github.

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127725177 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127673767 [Test build #39708 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39708/consoleFull) for PR 7774 at commit [`57d4cd2`](https://gith

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127672923 [Test build #206 has started](https://amplab.cs.berkeley.edu/jenkins/job/SlowSparkPullRequestBuilder/206/consoleFull) for PR 7774 at commit [`57d4cd2`](https://gith

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127672761 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127672704 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127580945 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127580739 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127580679 retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this fea

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127572833 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127572683 [Test build #39693 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39693/console) for PR 7774 at commit [`57d4cd2`](https://github.

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127565121 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127565052 [Test build #39690 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39690/console) for PR 7774 at commit [`cc1c736`](https://github.

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127534207 [Test build #39693 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39693/consoleFull) for PR 7774 at commit [`57d4cd2`](https://gith

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127533388 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127533378 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127532781 Review status: 5 of 26 files reviewed at latest revision, 71 unresolved discussions, all commit checks successful. --- **[sql/core/s

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127529681 @JoshRosen I ran into the following failure but it ran successfully if disable Tungsten. The input data has only 3 rows. ``` scala> val df = sqlContext.read

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127527279 Review status: 5 of 26 files reviewed at latest revision, 71 unresolved discussions, all commit checks successful. --- **[sql/core/

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127526731 [Test build #39690 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39690/consoleFull) for PR 7774 at commit [`cc1c736`](https://gith

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127523653 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127523595 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127508433 [Test build #39665 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39665/console) for PR 7774 at commit [`b8d5605`](https://github.

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127508472 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127506045 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127506003 [Test build #39658 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39658/console) for PR 7774 at commit [`4ed11a1`](https://github.

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127506005 [Test build #196 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SlowSparkPullRequestBuilder/196/console) for PR 7774 at commit [`4ed11a1`](https://github.

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127506046 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127496403 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127496359 **[Test build #39649 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39649/console)** for PR 7774 at commit [`bb52359`](https://github

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127496033 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127495919 [Test build #39652 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39652/console) for PR 7774 at commit [`332639c`](https://github.

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127486947 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127486922 **[Test build #39644 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39644/console)** for PR 7774 at commit [`7ab4816`](https://github

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/7774#discussion_r36158328 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/SparkPlan.scala --- @@ -59,12 +59,28 @@ abstract class SparkPlan extends QueryPlan[SparkPlan] w

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-12748 [Test build #39665 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39665/consoleFull) for PR 7774 at commit [`b8d5605`](https://gith

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127482141 Review status: 7 of 22 files reviewed at latest revision, 68 unresolved discussions, all commit checks successful. --- **[sql/core/

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127482143 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127482153 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127481808 Review status: 7 of 22 files reviewed at latest revision, 68 unresolved discussions, all commit checks successful. --- **[sql/core/

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127480754 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127480733 **[Test build #39638 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39638/console)** for PR 7774 at commit [`3a66207`](https://github

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127478631 [Test build #39642 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39642/console) for PR 7774 at commit [`dae195e`](https://github.

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127478749 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127477606 [Test build #39658 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39658/consoleFull) for PR 7774 at commit [`4ed11a1`](https://gith

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127476459 [Test build #196 has started](https://amplab.cs.berkeley.edu/jenkins/job/SlowSparkPullRequestBuilder/196/consoleFull) for PR 7774 at commit [`4ed11a1`](https://gith

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127476362 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127476387 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127476071 @JoshRosen https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39657/console looks the hash values of commits are wrong. --- If your project is set up f

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127475904 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127475972 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127475585 retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this fea

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127475475 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127475470 [Test build #39657 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39657/consoleFull) for PR 7774 at commit [`4ed11a1`](https://gith

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127475474 [Test build #39657 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39657/console) for PR 7774 at commit [`4ed11a1`](https://github.

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127475430 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not h

[GitHub] spark pull request: [SPARK-8861][SPARK-8862][SQL] Add basic instru...

2015-08-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7774#issuecomment-127475442 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

  1   2   3   >