[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user ueshin commented on the issue: https://github.com/apache/spark/pull/21155 Thanks! merging to master. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92334/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #92334 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92334/testReport)** for PR 21155 at commit [`d6de8cb`](https://github.com/apache/spark/commit/d6de8cb9b571f23dc357caf818295d359e19ff3d). * 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92331/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #92331 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92331/testReport)** for PR 21155 at commit [`140225d`](https://github.com/apache/spark/commit/140225d8b2326122f69e9093a071c1cdde7e489d). * 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #92334 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92334/testReport)** for PR 21155 at commit [`d6de8cb`](https://github.com/apache/spark/commit/d6de8cb9b571f23dc357caf818295d359e19ff3d). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #92331 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92331/testReport)** for PR 21155 at commit [`140225d`](https://github.com/apache/spark/commit/140225d8b2326122f69e9093a071c1cdde7e489d). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user ueshin commented on the issue: https://github.com/apache/spark/pull/21155 Jenkins, 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user wajda commented on the issue: https://github.com/apache/spark/pull/21155 please retest --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #92326 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92326/testReport)** for PR 21155 at commit [`140225d`](https://github.com/apache/spark/commit/140225d8b2326122f69e9093a071c1cdde7e489d). * This patch **fails due to an unknown error code, -9**. * 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92326/ Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #92326 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92326/testReport)** for PR 21155 at commit [`140225d`](https://github.com/apache/spark/commit/140225d8b2326122f69e9093a071c1cdde7e489d). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92295/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #92295 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92295/testReport)** for PR 21155 at commit [`def678b`](https://github.com/apache/spark/commit/def678b152b74e745b125167e55a2d03e239). * 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #92295 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92295/testReport)** for PR 21155 at commit [`def678b`](https://github.com/apache/spark/commit/def678b152b74e745b125167e55a2d03e239). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92217/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #92217 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92217/testReport)** for PR 21155 at commit [`c54857a`](https://github.com/apache/spark/commit/c54857a9cc9cdb64639913bc539975e2b507af28). * 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #92217 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92217/testReport)** for PR 21155 at commit [`c54857a`](https://github.com/apache/spark/commit/c54857a9cc9cdb64639913bc539975e2b507af28). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91617/ Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #91617 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91617/testReport)** for PR 21155 at commit [`2ccb9bb`](https://github.com/apache/spark/commit/2ccb9bb73f729c361ee819316e9c95224797a178). * This patch **fails to build**. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * `case class Sequence(` --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #91617 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91617/testReport)** for PR 21155 at commit [`2ccb9bb`](https://github.com/apache/spark/commit/2ccb9bb73f729c361ee819316e9c95224797a178). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/21155 ok to test --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user wajda commented on the issue: https://github.com/apache/spark/pull/21155 Any other comments on this one? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90909/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #90909 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90909/testReport)** for PR 21155 at commit [`2ccb9bb`](https://github.com/apache/spark/commit/2ccb9bb73f729c361ee819316e9c95224797a178). * This patch passes all tests. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * `case class Sequence(` --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90893/ Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #90893 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90893/testReport)** for PR 21155 at commit [`3ef9566`](https://github.com/apache/spark/commit/3ef9566dd85e49ecae74f888b576ba17e6e58f09). * This patch **fails SparkR unit tests**. * This patch **does not merge cleanly**. * This patch adds the following public classes _(experimental)_: * `case class Sequence(` --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90899/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #90899 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90899/testReport)** for PR 21155 at commit [`4b8af57`](https://github.com/apache/spark/commit/4b8af571d625d4c7c029757896f050001ab1e4f3). * This patch passes all tests. * This patch merges cleanly. * This patch adds the following public classes _(experimental)_: * `case class Sequence(` --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #90909 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90909/testReport)** for PR 21155 at commit [`2ccb9bb`](https://github.com/apache/spark/commit/2ccb9bb73f729c361ee819316e9c95224797a178). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #90899 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90899/testReport)** for PR 21155 at commit [`4b8af57`](https://github.com/apache/spark/commit/4b8af571d625d4c7c029757896f050001ab1e4f3). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #90892 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90892/testReport)** for PR 21155 at commit [`1b74d6f`](https://github.com/apache/spark/commit/1b74d6f734724b8e68b4f80c4e16f956cd19f92a). * This patch **fails Scala style 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90892/ Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #90893 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90893/testReport)** for PR 21155 at commit [`3ef9566`](https://github.com/apache/spark/commit/3ef9566dd85e49ecae74f888b576ba17e6e58f09). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #90892 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90892/testReport)** for PR 21155 at commit [`1b74d6f`](https://github.com/apache/spark/commit/1b74d6f734724b8e68b4f80c4e16f956cd19f92a). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90456/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #90456 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90456/testReport)** for PR 21155 at commit [`0a0a1e6`](https://github.com/apache/spark/commit/0a0a1e655f5b005d337a521692bbe9e6e09e5377). * 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #90456 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90456/testReport)** for PR 21155 at commit [`0a0a1e6`](https://github.com/apache/spark/commit/0a0a1e655f5b005d337a521692bbe9e6e09e5377). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90428/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #90428 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90428/testReport)** for PR 21155 at commit [`22bde31`](https://github.com/apache/spark/commit/22bde31feab95e548351a6057f5286c6faf75695). * 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #90428 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90428/testReport)** for PR 21155 at commit [`22bde31`](https://github.com/apache/spark/commit/22bde31feab95e548351a6057f5286c6faf75695). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90329/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #90329 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90329/testReport)** for PR 21155 at commit [`8786192`](https://github.com/apache/spark/commit/878619276592d189d69d9c6ed259f3d7d1f6fa48). * 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90330/ Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #90330 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90330/testReport)** for PR 21155 at commit [`383f180`](https://github.com/apache/spark/commit/383f1800470dbd7d54516e8f5609ee17dabb37ae). * 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #90330 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90330/testReport)** for PR 21155 at commit [`383f180`](https://github.com/apache/spark/commit/383f1800470dbd7d54516e8f5609ee17dabb37ae). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #90329 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90329/testReport)** for PR 21155 at commit [`8786192`](https://github.com/apache/spark/commit/878619276592d189d69d9c6ed259f3d7d1f6fa48). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90207/ Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21155 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #90207 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90207/testReport)** for PR 21155 at commit [`d15dad8`](https://github.com/apache/spark/commit/d15dad8ed169e91b12a4243ba275d9efcd680473). * 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 #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21155 **[Test build #90207 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90207/testReport)** for PR 21155 at commit [`d15dad8`](https://github.com/apache/spark/commit/d15dad8ed169e91b12a4243ba275d9efcd680473). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user wajda commented on the issue: https://github.com/apache/spark/pull/21155 Regarding ```step, stop, step``` vs. ```arg1, arg2, arg3```, ```left, middle, right``` and so on. Originally I used those to avoid hiding variables from the outer scope, but after the final refactoring it doesn't seem to be an issue anymore. I'll rename those, but I would keep ones as is for the ```def eval(input1: Any, input2: Any, input3: Any)``` method implementations. The reason is to distinguish the argument of type Any from their cast references as below: ``` override def eval(input1: Any, input2: Any, input3: Any): Array[T] = { val start = input1.asInstanceOf[T] val stop = input2.asInstanceOf[T] val step = input3.asInstanceOf[T] ``` --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark issue #21155: [SPARK-23927][SQL] Add "sequence" expression
Github user wajda commented on the issue: https://github.com/apache/spark/pull/21155 Thanks you for the comments. I'll review and make changes shortly. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org