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

    https://github.com/apache/spark/pull/19800#discussion_r152881639
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/GenerateOrdering.scala
 ---
    @@ -149,10 +151,14 @@ object GenerateOrdering extends 
CodeGenerator[Seq[SortOrder], Ordering[InternalR
           })
         // make sure INPUT_ROW is declared even if splitExpressions
         // returns an inlined block
    -    s"""
    +    val finalCode = s"""
            |InternalRow ${ctx.INPUT_ROW} = null;
            |$code
          """.stripMargin
    +    // Restore original currentVars and INPUT_ROW.
    +    ctx.currentVars = oldCurrentVars
    +    ctx.INPUT_ROW = oldInputRow
    +    finalCode
    --- End diff --
    
    Yes, thanks.


---

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

Reply via email to