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

    https://github.com/apache/spark/pull/21361#discussion_r189214135
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
 ---
    @@ -504,6 +504,9 @@ case class ArrayContains(left: Expression, right: 
Expression)
     
       override def dataType: DataType = BooleanType
     
    +  @transient private lazy val ordering: Ordering[Any] =
    +    TypeUtils.getInterpretedOrdering(right.dataType)
    --- End diff --
    
    Then in `checkInputDataTypes` we should check if there is `Ordering` for 
`right.dataType`. Otherwise for example MapType will throw a match error.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to