Github user mn-mikke commented on a diff in the pull request:

    https://github.com/apache/spark/pull/21103#discussion_r206228544
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
 ---
    @@ -3651,14 +3651,9 @@ case class ArrayDistinct(child: Expression)
     }
     
     /**
    - * Will become common base class for [[ArrayUnion]], ArrayIntersect, and 
ArrayExcept.
    + * Will become common base class for [[ArrayUnion]], ArrayIntersect, and 
[[ArrayExcept]].
      */
     abstract class ArraySetLike extends BinaryArrayExpressionWithImplicitCast {
    --- End diff --
    
    I'm not sure to what level Presto is considered as a reference. Just FYI 
these operations in Presto can accept arrays of different types.
    ```
    presto:default> SELECT array_except(ARRAY[5, 1, 7], ARRAY[7.0, 1.0, 3.0]);
     _col0 
    -------
     [5.0] 
    (1 row)
    ```


---

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

Reply via email to