Skip to content

Commit

Permalink
[MINOR] Fix a typo in First aggregate expression
Browse files Browse the repository at this point in the history
  • Loading branch information
exmy committed Sep 30, 2024
1 parent 039fd13 commit 924c77b
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@ case class First(child: Expression, ignoreNulls: Boolean)

override lazy val mergeExpressions: Seq[Expression] = {
// For first, we can just check if valueSet.left is set to true. If it is set
// to true, we use first.right. If not, we use first.right (even if valueSet.right is
// to true, we use first.left. If not, we use first.right (even if valueSet.right is
// false, we are safe to do so because first.right will be null in this case).
Seq(
/* first = */ If(valueSet.left, first.left, first.right),
Expand Down

0 comments on commit 924c77b

Please sign in to comment.