Github user mgaido91 commented on a diff in the pull request:

    https://github.com/apache/spark/pull/21045#discussion_r189177255
  
    --- Diff: 
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CollectionExpressionsSuite.scala
 ---
    @@ -265,6 +266,56 @@ class CollectionExpressionsSuite extends SparkFunSuite 
with ExpressionEvalHelper
           Some(Literal.create(null, StringType))), null)
       }
     
    +  test("Zip") {
    +    val literals = Seq(
    +      Literal.create(Seq(9001, 9002, 9003, null), ArrayType(IntegerType)),
    +      Literal.create(Seq(null, 1L, null, 4L, 11L), ArrayType(LongType)),
    +      Literal.create(Seq(-1, -3, 900, null), ArrayType(IntegerType)),
    +      Literal.create(Seq("a", null, "c"), ArrayType(StringType)),
    +      Literal.create(Seq(null, false, true), ArrayType(BooleanType)),
    +      Literal.create(Seq(1.1, null, 1.3, null), ArrayType(DoubleType)),
    +      Literal.create(Seq(), ArrayType(NullType)),
    +      Literal.create(Seq(null), ArrayType(NullType)),
    +      Literal.create(Seq(192.toByte), ArrayType(ByteType))
    --- End diff --
    
    Can we ave also an array of binary? And an array of arrays?


---

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

Reply via email to