[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-08 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/18567 Thank you for review and merging, @cloud-fan and @gatorsmile . Now, the dashboard looks good. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-08 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/18567 thanks, merging to master/2.2! --- 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

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18567 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 does not have this feature

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18567 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79375/ Test PASSed. ---

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-08 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18567 **[Test build #79375 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79375/testReport)** for PR 18567 at commit

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-08 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18567 **[Test build #79375 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79375/testReport)** for PR 18567 at commit

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-08 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/18567 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 feature enabled and

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18567 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79369/ Test FAILed. ---

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-08 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18567 **[Test build #79370 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79370/testReport)** for PR 18567 at commit

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18567 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 does not have this feature

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18567 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 does not have this feature

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18567 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79370/ Test FAILed. ---

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-08 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18567 **[Test build #79369 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79369/testReport)** for PR 18567 at commit

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-08 Thread cloud-fan
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/18567 LGTM, pending 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 not have this feature enabled and wishes

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-08 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18567 **[Test build #79370 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79370/testReport)** for PR 18567 at commit

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-07 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18567 **[Test build #79369 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79369/testReport)** for PR 18567 at commit

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-07 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/18567 Now, each unit test becomes independent. --- 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 issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18567 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79356/ Test PASSed. ---

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18567 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 does not have this feature

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-07 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18567 **[Test build #79356 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79356/testReport)** for PR 18567 at commit

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-07 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18567 **[Test build #79356 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79356/testReport)** for PR 18567 at commit

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-07 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/18567 I think we can add `beforeAll` with another `BeforeAndAfterAll`, but I tried to clean up instead of `assert` first. If this is too radical, I'll add `beforeAll`. --- If your project is set

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-07 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/18567 Oops. I tested the last commit by runing this suite only. ``` SparkSessionBuilderSuite: - create with config options and propagate them to SparkContext and SparkSession

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18567 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 does not have this feature

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-07 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18567 **[Test build #79352 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79352/testReport)** for PR 18567 at commit

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18567 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79352/ Test FAILed. ---

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-07 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/18567 LGTM pending Jenkins --- 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

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-07 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/18567 The failure is irrelevant to this PR. We can see the latest running Jenkins #79352 . ``` *** RUN ABORTED *** java.io.IOException: Failed to delete:

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18567 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 does not have this feature

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18567 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79351/ Test FAILed. ---

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-07 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18567 **[Test build #79351 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79351/testReport)** for PR 18567 at commit

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-07 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18567 **[Test build #79352 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79352/testReport)** for PR 18567 at commit

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-07 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/18567 According to the log of #18569, the sanity check fails with the same error as expected. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-07 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the issue: https://github.com/apache/spark/pull/18567 For safety, I added assertion for every cases in this test suite and found one line was missed. ```scala assert(SparkSession.getDefaultSession === None)

[GitHub] spark issue #18567: [SPARK-21345][SQL][TEST][test-maven] SparkSessionBuilder...

2017-07-07 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/18567 **[Test build #79351 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79351/testReport)** for PR 18567 at commit