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

    https://github.com/apache/spark/pull/20008#discussion_r157920327
  
    --- Diff: 
sql/core/src/test/resources/sql-tests/inputs/typeCoercion/native/functionArgumentConversion.sql
 ---
    @@ -25,7 +25,7 @@ SELECT array(cast(1 as tinyint), cast(1 as float)) FROM t;
     SELECT array(cast(1 as tinyint), cast(1 as double)) FROM t;
     SELECT array(cast(1 as tinyint), cast(1 as decimal(10, 0))) FROM t;
     SELECT array(cast(1 as tinyint), cast(1 as string)) FROM t;
    -SELECT array(cast(1 as tinyint), cast('1' as binary)) FROM t;
    +SELECT size(array(cast(1 as tinyint), cast('1' as binary))) FROM t;
    --- End diff --
    
    Replace `array(cast(1 as tinyint), cast('1' as binary))` with 
`size(array(cast(1 as tinyint), cast('1' as binary)))` to avoid binary type 
with collection.


---

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

Reply via email to