Github user cloud-fan commented on a diff in the pull request:

    https://github.com/apache/spark/pull/18516#discussion_r216292172
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeCreator.scala
 ---
    @@ -292,14 +296,17 @@ trait CreateNamedStructLike extends Expression {
       }
     
       override def checkInputDataTypes(): TypeCheckResult = {
    -    if (children.size % 2 != 0) {
    +    if (children.length < 1) {
    +      TypeCheckResult.TypeCheckFailure(
    +        s"input to function $prettyName requires at least one argument")
    --- End diff --
    
    This is not related to what this PR claims to do. What's the reason behind 
this change?


---

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

Reply via email to