[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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 commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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/65146730d72ece95681682582cd2ade9f6ae5edc). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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 commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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/65146730d72ece95681682582cd2ade9f6ae5edc). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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 commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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/b1f59f115302788b36743cd08b16b8b8a6a28027). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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 commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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/f840ffa5268268ace2e14d24f197194b98845bba). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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 commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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/b1f59f115302788b36743cd08b16b8b8a6a28027). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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/ Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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 commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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/f840ffa5268268ace2e14d24f197194b98845bba). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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.get moving forward. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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 commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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/8f3cbf3399420a14f5ebe74b99b2739437fe3647). * This patch **fails Spark unit tests**. * This patch merges cleanly. * This patch adds no public classes. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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 commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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/8f3cbf3399420a14f5ebe74b99b2739437fe3647). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20927: [SPARK-23809][SQL] Active SparkSession should be set by ...
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-h...@spark.apache.org