Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21361#discussion_r189262114
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -520,7 +523,12 @@ case class ArrayContains(left: Expression, right:
Expression)
TypeCheckResult.TypeCheckFailure(
"Arguments must be an array followed by a value of same type as
the array members")
} else {
- TypeCheckResult.TypeCheckSuccess
+ if (RowOrdering.isOrderable(right.dataType)) {
--- End diff --
we can call `TypeUtils.checkForOrderingExpr`
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
