[GitHub] spark issue #22789: [SPARK-25767][SQL] Fix lazily evaluated stream of expres...

2018-10-29 Thread peter-toth
Github user peter-toth commented on the issue: https://github.com/apache/spark/pull/22789 Thanks @mgaido91 @hvanhovell for the review. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #22789: [SPARK-25767][SQL] Fix lazily evaluated stream of expres...

2018-10-29 Thread hvanhovell
Github user hvanhovell commented on the issue: https://github.com/apache/spark/pull/22789 Merged to master/2.4 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: revie

[GitHub] spark issue #22789: [SPARK-25767][SQL] Fix lazily evaluated stream of expres...

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

[GitHub] spark issue #22789: [SPARK-25767][SQL] Fix lazily evaluated stream of expres...

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

[GitHub] spark issue #22789: [SPARK-25767][SQL] Fix lazily evaluated stream of expres...

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

[GitHub] spark issue #22789: [SPARK-25767][SQL] Fix lazily evaluated stream of expres...

2018-10-28 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/22789 **[Test build #98172 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98172/testReport)** for PR 22789 at commit [`3608f74`](https://github.com/apache/spark/commit/3

[GitHub] spark issue #22789: [SPARK-25767][SQL] Fix lazily evaluated stream of expres...

2018-10-28 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/22789 **[Test build #98172 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98172/testReport)** for PR 22789 at commit [`3608f74`](https://github.com/apache/spark/commit/36

[GitHub] spark issue #22789: [SPARK-25767][SQL] Fix lazily evaluated stream of expres...

2018-10-28 Thread hvanhovell
Github user hvanhovell commented on the issue: https://github.com/apache/spark/pull/22789 ok to test --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@sp

[GitHub] spark issue #22789: [SPARK-25767][SQL] Fix lazily evaluated stream of expres...

2018-10-28 Thread mgaido91
Github user mgaido91 commented on the issue: https://github.com/apache/spark/pull/22789 I don't have other comments @peter-toth. Maybe a committer can trigger a test for this. --- - To unsubscribe, e-mail: reviews-u

[GitHub] spark issue #22789: [SPARK-25767][SQL] Fix lazily evaluated stream of expres...

2018-10-28 Thread peter-toth
Github user peter-toth commented on the issue: https://github.com/apache/spark/pull/22789 @kiszk , @mgaido91, @hvanhovell anything I can add to this PR? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.