[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-04-03 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/20927 Thanks! Merged to master --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: r

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-04-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20927 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional comma

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

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

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-04-03 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20927 **[Test build #88856 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88856/testReport)** for PR 20927 at commit [`6514673`](https://github.com/apache/spark/commit/6

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-04-03 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/20927 LGTM pending Jenkins --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: revi

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-04-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20927 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional comma

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-04-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20927 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1928/ Tes

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-04-03 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20927 **[Test build #88856 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88856/testReport)** for PR 20927 at commit [`6514673`](https://github.com/apache/spark/commit/65

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-04-03 Thread ericl
Github user ericl commented on the issue: https://github.com/apache/spark/pull/20927 @gatorsmile done! --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-03-29 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/20927 @ericl Address the conflict? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mai

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

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

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-03-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20927 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional comma

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-03-29 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20927 **[Test build #88724 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88724/testReport)** for PR 20927 at commit [`b1f59f1`](https://github.com/apache/spark/commit/b

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

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

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-03-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20927 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional comma

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-03-29 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20927 **[Test build #88723 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88723/testReport)** for PR 20927 at commit [`f840ffa`](https://github.com/apache/spark/commit/f

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-03-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20927 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1846/ Tes

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-03-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20927 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional comma

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-03-29 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20927 **[Test build #88724 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88724/testReport)** for PR 20927 at commit [`b1f59f1`](https://github.com/apache/spark/commit/b1

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-03-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20927 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1845/ Tes

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-03-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20927 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional comma

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-03-29 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20927 **[Test build #88723 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88723/testReport)** for PR 20927 at commit [`f840ffa`](https://github.com/apache/spark/commit/f8

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-03-29 Thread ericl
Github user ericl commented on the issue: https://github.com/apache/spark/pull/20927 Per @marmbrus 's suggestion also added a `SparkSession.active` call to cover any edge case where the thread-local might not be set correctly. Spark code should prefer to use this instead of getActive.

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-03-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20927 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional comma

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

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

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-03-28 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20927 **[Test build #88681 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88681/testReport)** for PR 20927 at commit [`8f3cbf3`](https://github.com/apache/spark/commit/8

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-03-28 Thread gatorsmile
Github user gatorsmile commented on the issue: https://github.com/apache/spark/pull/20927 LGTM --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.ap

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-03-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20927 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional comma

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-03-28 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20927 **[Test build #88681 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88681/testReport)** for PR 20927 at commit [`8f3cbf3`](https://github.com/apache/spark/commit/8f

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-03-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20927 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/1814/ Tes

[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...

2018-03-28 Thread ericl
Github user ericl commented on the issue: https://github.com/apache/spark/pull/20927 @gatorsmile @marmbrus --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-