Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/20771#discussion_r173231642
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -599,8 +610,71 @@ case class MapObjects
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20772
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88094/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20772
**[Test build #88094 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88094/testReport)**
for PR 20772 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20772
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20773
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88095/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20773
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20773
**[Test build #88095 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88095/testReport)**
for PR 20773 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20753
**[Test build #88100 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88100/testReport)**
for PR 20753 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20756
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20756
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88092/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
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/1406/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20756
**[Test build #88092 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88092/testReport)**
for PR 20756 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88089/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
Build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20756#discussion_r173224239
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -1261,8 +1261,39 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20753
**[Test build #88089 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88089/testReport)**
for PR 20753 at commit
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/20772
Yeah, thank you for your explanation. Good catch.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20525
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20525
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/1405/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20773
**[Test build #88098 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88098/testReport)**
for PR 20773 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20525
**[Test build #88099 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88099/testReport)**
for PR 20525 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20773
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/1404/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20773
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20773#discussion_r173217810
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/MiscExpressionsSuite.scala
---
@@ -47,18 +47,24 @@ class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88097/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20753
**[Test build #88097 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88097/testReport)**
for PR 20753 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
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/1403/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20773
**[Test build #88096 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88096/testReport)**
for PR 20773 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20753
**[Test build #88097 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88097/testReport)**
for PR 20753 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20773
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20773
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/1402/
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/20773#discussion_r173216739
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/MiscExpressionsSuite.scala
---
@@ -47,18 +47,24 @@ class
Github user brad-kaiser commented on the issue:
https://github.com/apache/spark/pull/19041
Hi @squito , thank you for your feedback! I have not been able to work on
this PR lately, but I will get back to it soon. @vanzin I will also address the
rest of your feedback and fix those
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/20295
Thanks all for review! @HyukjinKwon do you mean this doc?
https://github.com/apache/spark/blob/master/docs/sql-programming-guide.md#pyspark-usage-guide-for-pandas-with-apache-arrow
I
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/20772
@kiszk the logic is not changed, but the previous implementation is wrong.
Let's go through an example to show more clearly this. Let's assume we have a
long nullable variable. Substituting the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20771
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20771
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88087/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20771
**[Test build #88087 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88087/testReport)**
for PR 20771 at commit
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/20773#discussion_r173199589
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/MiscExpressionsSuite.scala
---
@@ -43,4 +45,21 @@ class
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/20772
I may be wrong, but I do not find algorithm changes in
`calculateParamLength`.
I believe that the original implementation is correct. This is because
`calculateParamLength` returns the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20757
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88088/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20757
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20757
**[Test build #88088 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88088/testReport)**
for PR 20757 at commit
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/20757#discussion_r173190616
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/ScalaReflection.scala
---
@@ -121,6 +121,19 @@ object ScalaReflection extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88093/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20753
**[Test build #88093 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88093/testReport)**
for PR 20753 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20688
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20688
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88086/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20688
**[Test build #88086 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88086/testReport)**
for PR 20688 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20773
**[Test build #88095 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88095/testReport)**
for PR 20773 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20773
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/1401/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20773
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user mgaido91 opened a pull request:
https://github.com/apache/spark/pull/20773
[SPARK-23602][SQL] PrintToStderr prints value also in interpreted mode
## What changes were proposed in this pull request?
`PrintToStderr` was doing what is it supposed to only when code
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20688
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88085/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20688
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20688
**[Test build #88085 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88085/testReport)**
for PR 20688 at commit
Github user matyix commented on the issue:
https://github.com/apache/spark/pull/19775
@GaalDornick @erikerlandson @jerryshao @felixcheung et all
We gave up this - we have made the requested changes several times and I am
not willing to put more time on this and get in the
Github user matyix closed the pull request at:
https://github.com/apache/spark/pull/19775
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20772
**[Test build #88094 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88094/testReport)**
for PR 20772 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20772
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/1400/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20772
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/20772
cc @cloud-fan @kiszk @viirya
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
GitHub user mgaido91 opened a pull request:
https://github.com/apache/spark/pull/20772
[SPARK-23628][SQL] calculateParamLength should not return 1 + num of
epressions
## What changes were proposed in this pull request?
There was a bug in `calculateParamLength` which caused
Github user GaalDornick commented on the issue:
https://github.com/apache/spark/pull/19775
So, where did we land on the discussion for this PR. Is this change getting
in or is it out?
---
-
To unsubscribe, e-mail:
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20472
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20472
Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20750
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20750
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88084/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20750
**[Test build #88084 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88084/testReport)**
for PR 20750 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
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/1399/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20753
**[Test build #88093 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88093/testReport)**
for PR 20753 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20756
**[Test build #88092 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88092/testReport)**
for PR 20756 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20756
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/1398/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20756
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20753
**[Test build #88091 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88091/testReport)**
for PR 20753 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88091/
Test FAILed.
---
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/20757#discussion_r173159952
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/ScalaReflection.scala
---
@@ -121,6 +121,19 @@ object ScalaReflection extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20753
**[Test build #88091 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88091/testReport)**
for PR 20753 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
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/1397/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20756#discussion_r173159730
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -1261,8 +1261,42 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20753
**[Test build #88090 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88090/testReport)**
for PR 20753 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88090/
Test FAILed.
---
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20753#discussion_r173159210
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -188,8 +189,32 @@ case class StaticInvoke(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20753
**[Test build #88090 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88090/testReport)**
for PR 20753 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20753#discussion_r173158406
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ObjectExpressionsSuite.scala
---
@@ -71,6 +79,66 @@ class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20753
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/1396/
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20756#discussion_r173158104
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -1261,8 +1261,39 @@ case class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/20771#discussion_r173156834
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -501,12 +502,22 @@ case class
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20756#discussion_r173156685
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -1261,8 +1261,39 @@ case class
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20673#discussion_r173156595
--- Diff: core/src/main/scala/org/apache/spark/util/JsonProtocol.scala ---
@@ -100,7 +102,24 @@ private[spark] object JsonProtocol {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20753
**[Test build #88089 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/88089/testReport)**
for PR 20753 at commit
201 - 300 of 407 matches
Mail list logo