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

    https://github.com/apache/spark/pull/22315#discussion_r214532578
  
    --- Diff: 
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CollectionExpressionsSuite.scala
 ---
    @@ -383,10 +383,13 @@ class CollectionExpressionsSuite extends 
SparkFunSuite with ExpressionEvalHelper
         val a3 = Literal.create(null, ArrayType(StringType))
         val a4 = Literal.create(Seq(create_row(1)), ArrayType(StructType(Seq(
           StructField("a", IntegerType, true)))))
    +    // Explicitly mark the array type not nullable (spark-xxxxx)
    --- End diff --
    
    nit: `spark-xxxxx`?


---

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

Reply via email to