Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21074#discussion_r181620108
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -178,7 +178,13 @@ object TypeCoercion {
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21024
LGTM. Thanks! Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21024
---
-
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/21025
@mgaido91 Could you address the conflicts? Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21031#discussion_r181621963
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -3282,6 +3282,14 @@ object functions {
*/
def size(e:
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/20611
@wzhfy @HyukjinKwon @jiangxb1987 @gatorsmile @vinodkc
All test-cases has been passed, please let me know for any other
suggestions.
With current solution following improvement has
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20535
**[Test build #89382 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89382/testReport)**
for PR 20535 at commit
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21037#discussion_r181614224
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -287,3 +287,60 @@ case class
Github user edwinalu commented on a diff in the pull request:
https://github.com/apache/spark/pull/20940#discussion_r181611071
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/EventLoggingListener.scala ---
@@ -234,8 +244,22 @@ private[spark] class EventLoggingListener(
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/21047
Jenkins, retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21047
**[Test build #89384 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89384/testReport)**
for PR 21047 at commit
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21037#discussion_r181616266
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CollectionExpressionsSuite.scala
---
@@ -105,4 +106,26 @@ class
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21037#discussion_r181613044
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -287,3 +287,60 @@ case class
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21037#discussion_r181613270
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -287,3 +287,60 @@ case class
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21037#discussion_r181615751
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -287,3 +287,60 @@ case class
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21037#discussion_r181614388
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -287,3 +287,61 @@ case class
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21047
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21074#discussion_r181619518
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -178,7 +178,13 @@ object TypeCoercion {
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21053#discussion_r181619426
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -287,3 +287,106 @@ case class
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21053#discussion_r181617303
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeExtractors.scala
---
@@ -354,3 +336,37 @@ case class
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21053#discussion_r181619450
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -287,3 +287,106 @@ case class
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21053#discussion_r181618055
--- Diff: python/pyspark/sql/functions.py ---
@@ -1846,6 +1846,27 @@ def array_contains(col, value):
return
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21053#discussion_r181623803
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -287,3 +287,106 @@ case class
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21053#discussion_r181623729
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -287,3 +287,106 @@ case class
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21053#discussion_r181624253
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -287,3 +287,106 @@ case class
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/21033#discussion_r181627739
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackend.scala
---
@@ -495,9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21053
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89383/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21053
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/21053
**[Test build #89383 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89383/testReport)**
for PR 21053 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20535
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/20535
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/2333/
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21053#discussion_r181610494
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameFunctionsSuite.scala ---
@@ -413,6 +413,78 @@ class DataFrameFunctionsSuite extends QueryTest
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21053
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/2334/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21053
**[Test build #89383 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89383/testReport)**
for PR 21053 at commit
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21074
cc @HyukjinKwon @gatorsmile @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/21047
Merging to master. Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21031#discussion_r181622107
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/FunctionRegistry.scala
---
@@ -407,6 +407,7 @@ object FunctionRegistry {
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21053#discussion_r181622879
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -287,3 +287,106 @@ case class
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21053#discussion_r181623519
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -287,3 +287,106 @@ case class
101 - 139 of 139 matches
Mail list logo