[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user viirya commented on the issue: https://github.com/apache/spark/pull/20976 ah, a late LGTM. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/20976 thanks, merging to master/2.3! --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89444/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 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 #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20976 **[Test build #89444 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89444/testReport)** for PR 20976 at commit [`92cd4fe`](https://github.com/apache/spark/commit/92cd4fef2442c7e80db85bf563200e2166d4bab4). * 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 #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 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/2384/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 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 #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20976 **[Test build #89444 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89444/testReport)** for PR 20976 at commit [`92cd4fe`](https://github.com/apache/spark/commit/92cd4fef2442c7e80db85bf563200e2166d4bab4). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user cloud-fan commented on the issue: https://github.com/apache/spark/pull/20976 LGTM, can you add an end-to-end test case in `DatasetSuite`? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user mgaido91 commented on the issue: https://github.com/apache/spark/pull/20976 kindly ping @cloud-fan @marmbrus @viirya --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user mgaido91 commented on the issue: https://github.com/apache/spark/pull/20976 cc @cloud-fan @jkbradley @marmbrus @viirya --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 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 #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88975/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20976 **[Test build #88975 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88975/testReport)** for PR 20976 at commit [`03c9313`](https://github.com/apache/spark/commit/03c931384d027480feb81818044e03d3871d2518). * 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 #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 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/2033/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 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 #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20976 **[Test build #88975 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88975/testReport)** for PR 20976 at commit [`03c9313`](https://github.com/apache/spark/commit/03c931384d027480feb81818044e03d3871d2518). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 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 #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88949/ Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20976 **[Test build #88949 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88949/testReport)** for PR 20976 at commit [`b78ff6c`](https://github.com/apache/spark/commit/b78ff6c480f25a080586402fcea51ce5732fa9ac). * 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 #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 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 #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 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/2011/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20976 **[Test build #88949 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88949/testReport)** for PR 20976 at commit [`b78ff6c`](https://github.com/apache/spark/commit/b78ff6c480f25a080586402fcea51ce5732fa9ac). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user mgaido91 commented on the issue: https://github.com/apache/spark/pull/20976 UT errors were caused by another commit which has been reverted. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user mgaido91 commented on the issue: https://github.com/apache/spark/pull/20976 retest this please --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 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 #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88940/ Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20976 **[Test build #88940 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88940/testReport)** for PR 20976 at commit [`b78ff6c`](https://github.com/apache/spark/commit/b78ff6c480f25a080586402fcea51ce5732fa9ac). * 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 #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 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/2004/ Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 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 #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20976 **[Test build #88940 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88940/testReport)** for PR 20976 at commit [`b78ff6c`](https://github.com/apache/spark/commit/b78ff6c480f25a080586402fcea51ce5732fa9ac). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88903/ Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 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 #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20976 **[Test build #88903 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88903/testReport)** for PR 20976 at commit [`b3e70e9`](https://github.com/apache/spark/commit/b3e70e95069a704abce88313559e4de686768e78). * 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 #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 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 #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 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/1972/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20976 **[Test build #88903 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88903/testReport)** for PR 20976 at commit [`b3e70e9`](https://github.com/apache/spark/commit/b3e70e95069a704abce88313559e4de686768e78). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user mgaido91 commented on the issue: https://github.com/apache/spark/pull/20976 retest this please --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 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 #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88892/ Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20976 **[Test build #88892 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88892/testReport)** for PR 20976 at commit [`b3e70e9`](https://github.com/apache/spark/commit/b3e70e95069a704abce88313559e4de686768e78). * 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 #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 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 #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20976 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/1960/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #20976: [SPARK-23835][SQL] Add not-null check to Tuples' argumen...
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20976 **[Test build #88892 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88892/testReport)** for PR 20976 at commit [`b3e70e9`](https://github.com/apache/spark/commit/b3e70e95069a704abce88313559e4de686768e78). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
