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

    https://github.com/apache/spark/pull/22029#discussion_r229692081
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala
 ---
    @@ -202,7 +225,11 @@ case class InSubquery(values: Seq[Expression], query: 
ListQuery)
      */
     // scalastyle:off line.size.limit
     @ExpressionDescription(
    -  usage = "expr1 _FUNC_(expr2, expr3, ...) - Returns true if `expr` equals 
to any valN.",
    +  usage = """
    +    expr1 _FUNC_(expr2, expr3, ...) - Returns true if `expr` equals to any 
valN. Otherwise, if
    +      spark.sql.legacy.inOperator.falseForNullField is false and any of 
the elements or fields of
    --- End diff --
    
    `any of the elements or fields ...`
    
    We should explicitly mention multi-column IN, which is different from `a in 
(b, c, ...)` while `a` is struct type.


---

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

Reply via email to