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

    https://github.com/apache/spark/pull/19733#discussion_r150547684
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/predicates.scala
 ---
    @@ -236,24 +236,38 @@ case class In(value: Expression, list: 
Seq[Expression]) extends Predicate {
       override def doGenCode(ctx: CodegenContext, ev: ExprCode): ExprCode = {
         val valueGen = value.genCode(ctx)
         val listGen = list.map(_.genCode(ctx))
    +    ctx.addMutableState("boolean", ev.value, "")
    +    ctx.addMutableState("boolean", ev.isNull, "")
    +    val valueArg = ctx.freshName("valueArg")
    --- End diff --
    
    Oh, sorry, I wasn't aware that `valueGen.value` can contains things like 
that. Thanks for the explanation.


---

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

Reply via email to