Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21193
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/21193
**[Test build #90174 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90174/testReport)**
for PR 21193 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21173
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/21173
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/2892/
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21193
Thanks for cc. I'll check tonight.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21220
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/2891/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21220
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/21061
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/21061
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/2890/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17332
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21173
**[Test build #90173 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90173/testReport)**
for PR 21173 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21220
**[Test build #90172 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90172/testReport)**
for PR 21220 at commit
Github user aditkumar commented on the issue:
https://github.com/apache/spark/pull/20787
Not at all
On Thu, May 3, 2018 at 9:30 PM Hyukjin Kwon
wrote:
> *@HyukjinKwon* commented on this pull request.
> --
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21229
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21224
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21229
Test FAILed.
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/2889/
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21193
I think this can be reviewed for now. Thanks. cc @cloud-fan @hvanhovell
@kiszk @maropu
---
-
To unsubscribe, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21224
and branch 2-3 too ..?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20787#discussion_r185976200
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/DateTimeUtils.scala
---
@@ -887,11 +887,25 @@ object DateTimeUtils {
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21224
thanks, merging to master!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21229
**[Test build #90170 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90170/testReport)**
for PR 21229 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21061
**[Test build #90171 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90171/testReport)**
for PR 21061 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20787
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90167/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20787
**[Test build #90167 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90167/testReport)**
for PR 20787 at commit
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21231
@hvanhovell @viirya @kiszk
---
-
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/20787
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21224#discussion_r185975883
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFileFormat.scala
---
@@ -342,6 +342,7 @@ class
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21229
retest this please
---
-
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/21193
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/21193
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/2888/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21193
**[Test build #90169 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90169/testReport)**
for PR 21193 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21222
**[Test build #90168 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90168/testReport)**
for PR 21222 at commit
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/21220#discussion_r185975133
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MicroBatchExecution.scala
---
@@ -128,40 +130,49 @@ class MicroBatchExecution(
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21222
add to whitelist
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/21220#discussion_r185974949
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MicroBatchExecution.scala
---
@@ -266,93 +276,62 @@ class MicroBatchExecution(
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/21220#discussion_r185974962
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MicroBatchExecution.scala
---
@@ -128,40 +130,49 @@ class MicroBatchExecution(
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21207
@arunmahadevan, not a big deal but mind if I ask to fix the PR title to
`[SPARK-24136][SS] blabla`? It's actually encouraged in the guide -
https://spark.apache.org/contributing.html
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20787
**[Test build #90167 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90167/testReport)**
for PR 20787 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20787
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21070
aha, I see and it might to be true. I'll check the benchmark again.
---
-
To unsubscribe, e-mail:
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/21231#discussion_r185974124
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/SortOrder.scala
---
@@ -147,7 +148,40 @@ case class SortPrefix(child:
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/21231#discussion_r185973696
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/SortOrder.scala
---
@@ -147,7 +148,40 @@ case class SortPrefix(child:
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/21220#discussion_r185973467
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/WatermarkTracker.scala
---
@@ -0,0 +1,75 @@
+/*
+ * Licensed to the Apache
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/21231#discussion_r185973474
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/SortOrder.scala
---
@@ -147,7 +148,40 @@ case class SortPrefix(child:
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/21220#discussion_r185973062
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MicroBatchExecution.scala
---
@@ -128,40 +130,49 @@ class MicroBatchExecution(
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/21220#discussion_r185972962
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MicroBatchExecution.scala
---
@@ -128,40 +130,49 @@ class MicroBatchExecution(
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/21070
@maropu, I suspect that the problem is that comparison is different for
strings: `"17297598712"` is less than `"5"` with string comparison.
---
Github user pepinoflo commented on a diff in the pull request:
https://github.com/apache/spark/pull/21208#discussion_r185972208
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1229,3 +1229,98 @@ case class
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21070
@rdblue Aha, thanks for the explanation!
> I think that you were expecting a string comparison case to have a
significant benefit over
> non->pushdown. But I would only expect that if
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16478
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/2887/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16478
Build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user pepinoflo commented on a diff in the pull request:
https://github.com/apache/spark/pull/21208#discussion_r185971775
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1229,3 +1229,132 @@ case class
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21218#discussion_r185970925
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/clustering/GaussianMixture.scala ---
@@ -423,6 +423,8 @@ class GaussianMixture @Since("2.0.0") (
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21225
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21225
Thanks! Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/21070
@gatorsmile, are you happy committing this with the benchmark results?
@maropu, thanks for taking the time to add these benchmarks, it is really
great to have them so we can monitor the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21135
**[Test build #90166 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90166/testReport)**
for PR 21135 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21135
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/21220#discussion_r185969637
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/WatermarkTracker.scala
---
@@ -0,0 +1,75 @@
+/*
+ * Licensed to the Apache
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21070
@rdblue ya, my bad for the simple scan case, you're right.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21225
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/21225
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90158/
Test PASSed.
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21169
ping @michal-databricks
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21225
**[Test build #90158 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90158/testReport)**
for PR 21225 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21187
---
-
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/21207
**[Test build #90165 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90165/testReport)**
for PR 21207 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21207
also ping @zsxwing
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21207
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21187
Thanks for your fast and great work! Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21231
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/21231
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90159/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21231
**[Test build #90159 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90159/testReport)**
for PR 21231 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21122#discussion_r185967201
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalog.scala
---
@@ -31,10 +30,16 @@ import
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/21207
@tdas @jerryshao @HyukjinKwon
Kindly ping to trigger test and review.
---
-
To unsubscribe, e-mail:
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/21222
@tdas @jose-torres @jerryshao @arunmahadevan
Kindly ping to review.
---
-
To unsubscribe, e-mail:
Github user pepinoflo commented on a diff in the pull request:
https://github.com/apache/spark/pull/21208#discussion_r185966094
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1229,3 +1229,98 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21208
**[Test build #90164 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90164/testReport)**
for PR 21208 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21021
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/2886/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21021
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/21021
**[Test build #90163 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90163/testReport)**
for PR 21021 at commit
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/21220#discussion_r185962542
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MicroBatchExecution.scala
---
@@ -266,93 +276,62 @@ class MicroBatchExecution(
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/21220#discussion_r185962261
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MicroBatchExecution.scala
---
@@ -128,40 +130,49 @@ class MicroBatchExecution(
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21208#discussion_r185962223
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1229,3 +1229,132 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21200
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user pepinoflo commented on a diff in the pull request:
https://github.com/apache/spark/pull/21208#discussion_r185962048
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1229,3 +1229,98 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21200
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90155/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21200
**[Test build #90155 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90155/testReport)**
for PR 21200 at commit
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/21220#discussion_r185961801
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MicroBatchExecution.scala
---
@@ -128,40 +130,49 @@ class MicroBatchExecution(
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/21220#discussion_r185961485
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MicroBatchExecution.scala
---
@@ -384,22 +363,21 @@ class MicroBatchExecution(
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/21220#discussion_r185961340
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MicroBatchExecution.scala
---
@@ -266,93 +276,62 @@ class MicroBatchExecution(
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/21220#discussion_r185961121
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/MicroBatchExecution.scala
---
@@ -266,93 +276,62 @@ class MicroBatchExecution(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21200
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/21200
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90154/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21200
**[Test build #90154 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90154/testReport)**
for PR 21200 at commit
Github user pepinoflo commented on a diff in the pull request:
https://github.com/apache/spark/pull/21208#discussion_r185958227
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1229,3 +1229,98 @@ case class
Github user pepinoflo commented on a diff in the pull request:
https://github.com/apache/spark/pull/21208#discussion_r185958060
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1229,3 +1229,98 @@ case class
Github user pepinoflo commented on a diff in the pull request:
https://github.com/apache/spark/pull/21208#discussion_r185958088
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1229,3 +1229,98 @@ case class
Github user pepinoflo commented on a diff in the pull request:
https://github.com/apache/spark/pull/21208#discussion_r185958070
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1229,3 +1229,98 @@ case class
Github user pepinoflo commented on a diff in the pull request:
https://github.com/apache/spark/pull/21208#discussion_r185958049
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1229,3 +1229,98 @@ case class
Github user pepinoflo commented on a diff in the pull request:
https://github.com/apache/spark/pull/21208#discussion_r185958014
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameFunctionsSuite.scala ---
@@ -798,6 +798,111 @@ class DataFrameFunctionsSuite extends
101 - 200 of 724 matches
Mail list logo