Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21061
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r201904049
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3261,3 +3261,322 @@ case class ArrayDistin
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r201898932
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3261,3 +3261,322 @@ case class ArrayDisti
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r201692534
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeArrayData.java
---
@@ -463,14 +463,27 @@ private static UnsafeArrayData
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r201688229
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3261,3 +3261,322 @@ case class ArrayDistin
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r201620967
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3261,3 +3261,322 @@ case class ArrayDisti
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r201618980
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeArrayData.java
---
@@ -463,14 +463,27 @@ private static UnsafeArrayData
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r201109190
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3261,3 +3261,323 @@ case class ArrayDistin
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200925450
--- Diff: python/pyspark/sql/functions.py ---
@@ -2013,6 +2013,25 @@ def array_distinct(col):
return Column(sc._jvm.functions.array_distinct(_to_java_
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200889057
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3261,3 +3261,323 @@ case class ArrayDisti
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200889598
--- Diff: python/pyspark/sql/functions.py ---
@@ -2013,6 +2013,25 @@ def array_distinct(col):
return Column(sc._jvm.functions.array_distinct(_to_java
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200888320
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3261,3 +3261,323 @@ case class ArrayDisti
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200887096
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3261,3 +3261,323 @@ case class ArrayDistin
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200886228
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeArrayData.java
---
@@ -450,7 +450,7 @@ public UnsafeArrayData copy() {
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200885976
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeArrayData.java
---
@@ -450,7 +450,7 @@ public UnsafeArrayData copy() {
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200884043
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeArrayData.java
---
@@ -450,7 +450,7 @@ public UnsafeArrayData copy() {
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200883268
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeArrayData.java
---
@@ -450,7 +450,7 @@ public UnsafeArrayData copy() {
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200875757
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeArrayData.java
---
@@ -450,7 +450,7 @@ public UnsafeArrayData copy() {
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r189432089
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1882,3 +1882,311 @@ case class ArrayRepea
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200878344
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3261,3 +3261,323 @@ case class ArrayDisti
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200875456
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeArrayData.java
---
@@ -463,14 +463,27 @@ private static UnsafeArrayData
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200875538
--- Diff: python/pyspark/sql/functions.py ---
@@ -2013,6 +2013,25 @@ def array_distinct(col):
return Column(sc._jvm.functions.array_distinct(_to_java
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200876039
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3261,3 +3261,323 @@ case class ArrayDisti
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200874571
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CollectionExpressionsSuite.scala
---
@@ -1166,4 +1166,88 @@ class Collection
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200874190
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3261,3 +3261,323 @@ case class ArrayDisti
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200874014
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3261,3 +3261,323 @@ case class ArrayDisti
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200572386
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3261,3 +3261,339 @@ case class ArrayDistin
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200541043
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3261,3 +3261,339 @@ case class ArrayDisti
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200543679
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3261,3 +3261,339 @@ case class ArrayDisti
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200541138
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3261,3 +3261,339 @@ case class ArrayDisti
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200540949
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3261,3 +3261,339 @@ case class ArrayDisti
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200540913
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3261,3 +3261,339 @@ case class ArrayDisti
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r200541020
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -3261,3 +3261,339 @@ case class ArrayDisti
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r197319579
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2355,3 +2355,347 @@ case class ArrayRemove
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r197235630
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2355,3 +2355,347 @@ case class ArrayRemove
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r197229189
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2355,3 +2355,347 @@ case class ArrayRemove
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r197229074
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2355,3 +2355,347 @@ case class ArrayRemove
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r197228219
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2355,3 +2355,347 @@ case class ArrayRemove
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r197004678
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2355,3 +2355,347 @@ case class ArrayRemov
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r197001921
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2355,3 +2355,347 @@ case class ArrayRemov
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r197005444
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2355,3 +2355,347 @@ case class ArrayRemov
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r197003678
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2355,3 +2355,347 @@ case class ArrayRemov
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r197004881
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2355,3 +2355,347 @@ case class ArrayRemov
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r197005083
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2355,3 +2355,347 @@ case class ArrayRemov
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r197004822
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2355,3 +2355,347 @@ case class ArrayRemov
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r197004944
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2355,3 +2355,347 @@ case class ArrayRemov
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r197004467
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2355,3 +2355,347 @@ case class ArrayRemov
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r196674440
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2355,3 +2355,297 @@ case class ArrayRemove
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r196671471
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2355,3 +2355,297 @@ case class ArrayRemove
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r196662595
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2355,3 +2355,297 @@ case class ArrayRemov
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r196621059
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2189,3 +2189,293 @@ case class ArrayRemove
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r196620839
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2189,3 +2189,293 @@ case class ArrayRemove
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r196614601
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2189,3 +2189,293 @@ case class ArrayRemove
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r194527119
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2189,3 +2189,293 @@ case class ArrayRemov
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r194516045
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2189,3 +2189,293 @@ case class ArrayRemov
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r194523480
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2189,3 +2189,293 @@ case class ArrayRemov
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r194520924
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2189,3 +2189,293 @@ case class ArrayRemov
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r194530787
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2189,3 +2189,293 @@ case class ArrayRemov
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r194520589
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2189,3 +2189,293 @@ case class ArrayRemov
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r194520120
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2189,3 +2189,293 @@ case class ArrayRemov
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r194517152
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2189,3 +2189,293 @@ case class ArrayRemov
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r192919335
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2189,3 +2189,302 @@ case class ArrayRemov
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r192546226
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2189,3 +2189,302 @@ case class ArrayRemove
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r192520463
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -2189,3 +2189,302 @@ case class ArrayRemov
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r192490355
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1882,3 +1882,311 @@ case class ArrayRepeat
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r192479966
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1882,3 +1882,311 @@ case class ArrayRepea
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r192340073
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1882,3 +1882,311 @@ case class ArrayRepeat
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r192336364
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1882,3 +1882,311 @@ case class ArrayRepeat
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r192331296
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1882,3 +1882,311 @@ case class ArrayRepeat
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r192330635
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1882,3 +1882,311 @@ case class ArrayRepeat
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r192254481
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1882,3 +1882,311 @@ case class ArrayRepea
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r192252991
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1882,3 +1882,311 @@ case class ArrayRepea
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r192250043
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1882,3 +1882,311 @@ case class ArrayRepea
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r192251150
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1882,3 +1882,311 @@ case class ArrayRepea
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r192253086
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1882,3 +1882,311 @@ case class ArrayRepea
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r192249752
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1882,3 +1882,311 @@ case class ArrayRepea
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r192250954
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1882,3 +1882,311 @@ case class ArrayRepea
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r192251238
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1882,3 +1882,311 @@ case class ArrayRepea
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r192254166
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -1882,3 +1882,311 @@ case class ArrayRepea
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r186283042
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -19,13 +19,42 @@ package org.apache.spark.s
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r183011714
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CollectionExpressionsSuite.scala
---
@@ -169,4 +169,45 @@ class CollectionEx
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r183005260
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CollectionExpressionsSuite.scala
---
@@ -169,4 +169,45 @@ class CollectionExp
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r182990028
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CollectionExpressionsSuite.scala
---
@@ -169,4 +169,45 @@ class CollectionEx
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r182988201
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CollectionExpressionsSuite.scala
---
@@ -169,4 +169,45 @@ class CollectionExp
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r182973891
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CollectionExpressionsSuite.scala
---
@@ -169,4 +169,45 @@ class CollectionEx
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r182971562
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CollectionExpressionsSuite.scala
---
@@ -169,4 +169,45 @@ class CollectionExp
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r182971159
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -505,3 +506,150 @@ case class ArrayMax(chil
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r182690674
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -417,3 +418,156 @@ case class ArrayMax(chil
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r182690065
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CollectionExpressionsSuite.scala
---
@@ -169,4 +169,45 @@ class CollectionExp
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r182689118
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -505,3 +506,150 @@ case class ArrayMax(chil
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r182672237
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -505,3 +506,150 @@ case class ArrayMax(chi
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r182671380
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CollectionExpressionsSuite.scala
---
@@ -169,4 +169,45 @@ class CollectionEx
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r182663898
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -505,3 +506,150 @@ case class ArrayMax(chi
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r182674039
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -417,3 +418,156 @@ case class ArrayMax(chi
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r182532583
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -417,3 +418,156 @@ case class ArrayMax(chil
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r182484698
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -417,3 +418,156 @@ case class ArrayMax(chil
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r182200622
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -417,3 +418,156 @@ case class ArrayMax(chil
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r181590066
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -287,3 +288,80 @@ case class ArrayContains(
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r181524647
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -287,3 +288,80 @@ case class ArrayContains
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21061#discussion_r181478560
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -287,3 +288,80 @@ case class ArrayContains(
1 - 100 of 103 matches
Mail list logo