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

    https://github.com/apache/spark/pull/19733#discussion_r150484563
  
    --- 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 --
    
    `valueGen.value` may have casted variable (i.e. (UTF8String)variable1). It 
cannot be passed as an argument.


---

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

Reply via email to