[GitHub] spark issue #20736: [SPARK-23586][SQL] Add interpreted execution to UnwrapOp...

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

[GitHub] spark issue #20736: [SPARK-23586][SQL] Add interpreted execution to UnwrapOp...

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

[GitHub] spark issue #20736: [SPARK-23586][SQL] Add interpreted execution to UnwrapOp...

2018-03-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20736 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/1277/

[GitHub] spark issue #20736: [SPARK-23586][SQL] Add interpreted execution to UnwrapOp...

2018-03-05 Thread hvanhovell
Github user hvanhovell commented on the issue: https://github.com/apache/spark/pull/20736 LGTM - pending jenkins --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #20736: [SPARK-23586][SQL] Add interpreted execution to UnwrapOp...

2018-03-05 Thread mgaido91
Github user mgaido91 commented on the issue: https://github.com/apache/spark/pull/20736 @hvanhovell thanks for your comments, let's say this is a good opportunity to improve test coverage too for it --- - To

[GitHub] spark issue #20736: [SPARK-23586][SQL] Add interpreted execution to UnwrapOp...

2018-03-05 Thread hvanhovell
Github user hvanhovell commented on the issue: https://github.com/apache/spark/pull/20736 @mgaido91 you picked this up phenomenally fast! It looks pretty good, let two minor comments. --- - To unsubscribe, e-mail:

[GitHub] spark issue #20736: [SPARK-23586][SQL] Add interpreted execution to UnwrapOp...

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

[GitHub] spark issue #20736: [SPARK-23586][SQL] Add interpreted execution to UnwrapOp...

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

[GitHub] spark issue #20736: [SPARK-23586][SQL] Add interpreted execution to UnwrapOp...

2018-03-05 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20736 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/1273/