Skip to content

Commit a39098d

Browse files
committed
fix ordering error
1 parent be84c4d commit a39098d

File tree

1 file changed

+3
-0
lines changed
  • sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen

1 file changed

+3
-0
lines changed

sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -884,6 +884,9 @@ class CodegenContext {
884884
innerClassToFunctions.update(key, f.functionName +:
885885
innerClassToFunctions.getOrElse(key, Seq.empty[String]))
886886
})
887+
// for performance reasons, the functions are prepended, instead of appended,
888+
// thus they are in reversed order
889+
innerClassToFunctions.transform { case (_, functions) => functions.reverse }
887890

888891
val argDefinitionString = arguments.map { case (t, name) => s"$t $name" }.mkString(", ")
889892
val argInvocationString = arguments.map(_._2).mkString(", ")

0 commit comments

Comments
 (0)