[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 
[`d899f44`](https://github.com/apache/spark/commit/d899f44b213da4e3e819a308a2442a9202901ad3).


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[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 commands, e-mail: reviews-h...@spark.apache.org



[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/
Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[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: reviews-h...@spark.apache.org



[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 unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[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: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[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 
[`804394b`](https://github.com/apache/spark/commit/804394b44e65142c0af3eb3dc19bab29ed13875d).


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[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 commands, e-mail: reviews-h...@spark.apache.org



[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/
Test PASSed.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org