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

    https://github.com/apache/spark/pull/19800#discussion_r152794103
  
    --- Diff: 
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/OrderingSuite.scala
 ---
    @@ -156,4 +156,13 @@ class OrderingSuite extends SparkFunSuite with 
ExpressionEvalHelper {
           assert(genOrdering.compare(rowB1, rowB2) < 0)
         }
       }
    +
    +  test("SPARK-22591: GenerateOrdering shouldn't change ctx.INPUT_ROW") {
    --- End diff --
    
    I'm working to support wholestage codegen when generating expression codes 
safe from 64k limit. When there is not `INPUT_ROW` in context but we wrongly 
set a `INPUT_ROW` value, a non-existing `InternalRow i` will be added into 
function parameters.



---

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

Reply via email to