[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2016-04-23 Thread yhuai
Github user yhuai closed the pull request at: https://github.com/apache/spark/pull/9702 --- 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 enabl

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2016-03-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-197117117 **[Test build #2642 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2642/consoleFull)** for PR 9702 at commit [`238e288`](https://gi

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2016-03-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-197113107 **[Test build #2642 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2642/consoleFull)** for PR 9702 at commit [`238e288`](https://git

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-12-16 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-165296680 it is. I just did not get a chance to fix it. --- 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 proje

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-12-15 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-164894408 @yhuai looks like this has a legit failure in python --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-12-15 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-164894151 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

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-12-15 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-164701028 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/4

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-12-15 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-164700855 **[Test build #47722 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47722/consoleFull)** for PR 9702 at commit [`238e288`](https://git

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-12-14 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-164674728 **[Test build #47722 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47722/consoleFull)** for PR 9702 at commit [`238e288`](https://gith

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-12-14 Thread 3ourroom
Github user 3ourroom commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-164673161 NAVER - http://www.naver.com/ 3ourr...@naver.com 님께 보내신 메일 이 다음과 같은 이유로 ì 

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-12-14 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-164672747 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

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-12-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-164625136 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-11735] [SQL] Add a check in the constru...

2015-12-14 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-164625138 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/4

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-12-14 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-164624960 **[Test build #47694 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47694/consoleFull)** for PR 9702 at commit [`238e288`](https://git

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-12-14 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-164605811 **[Test build #47694 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47694/consoleFull)** for PR 9702 at commit [`238e288`](https://gith

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-12-14 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-164604869 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

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-12-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-161159482 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-11735] [SQL] Add a check in the constru...

2015-12-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-161159484 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/4

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-12-01 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-161159402 **[Test build #47017 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47017/consoleFull)** for PR 9702 at commit [`238e288`](https://git

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-12-01 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-161138768 We probably need to fix python test. --- 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 n

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-12-01 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-161137649 **[Test build #47017 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47017/consoleFull)** for PR 9702 at commit [`238e288`](https://gith

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-12-01 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-161136160 Jenkins, 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 h

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-11-26 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-160045786 **[Test build #2121 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2121/consoleFull)** for PR 9702 at commit [`238e288`](https://gi

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-11-26 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-160032984 **[Test build #2121 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2121/consoleFull)** for PR 9702 at commit [`238e288`](https://git

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-11-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-158800720 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/4

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-11-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-158800719 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-11735] [SQL] Add a check in the constru...

2015-11-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-158800694 **[Test build #46494 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46494/consoleFull)** for PR 9702 at commit [`238e288`](https://git

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-11-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-158789470 **[Test build #46494 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/46494/consoleFull)** for PR 9702 at commit [`238e288`](https://gith

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-11-20 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-158609501 Changes LGTM - can you bring it up to date and fix the style violation? --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-11-20 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/9702#discussion_r45542907 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala --- @@ -78,6 +78,9 @@ class SQLContext private[sql]( } def this(sparkContext

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-11-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-156589834 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/4

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-11-13 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-156589769 **[Test build #45889 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45889/consoleFull)** for PR 9702 at commit [`ad22179`](https://git

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-11-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-156589833 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-11735] [SQL] Add a check in the constru...

2015-11-13 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-156553514 **[Test build #45889 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45889/consoleFull)** for PR 9702 at commit [`ad22179`](https://gith

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-11-13 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/9702#discussion_r44831686 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLContext.scala --- @@ -78,6 +78,9 @@ class SQLContext private[sql]( } def this(sparkContex

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-11-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-156549648 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-11735] [SQL] Add a check in the constru...

2015-11-13 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-156549650 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/4

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-11-13 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-156549613 **[Test build #45885 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45885/consoleFull)** for PR 9702 at commit [`197aaa3`](https://git

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-11-13 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/9702#issuecomment-156545901 **[Test build #45885 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45885/consoleFull)** for PR 9702 at commit [`197aaa3`](https://gith

[GitHub] spark pull request: [SPARK-11735] [SQL] Add a check in the constru...

2015-11-13 Thread yhuai
GitHub user yhuai opened a pull request: https://github.com/apache/spark/pull/9702 [SPARK-11735] [SQL] Add a check in the constructor of SQLContext to make sure its SparkContext is not stopped https://issues.apache.org/jira/browse/SPARK-11735 You can merge this pull request into a