Skip to content

Commit e30be7a

Browse files
committed
Address comments.
1 parent 2d77332 commit e30be7a

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeBlockSuite.scala

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ class CodeBlockSuite extends SparkFunSuite {
123123

124124
// We want to replace all occurrences of `expr` with the variable `aliasedParam`.
125125
val aliasedCode = code.transformExprValues {
126-
case SimpleExprValue("1 + 1", _) => aliasedParam
126+
case SimpleExprValue("1 + 1", java.lang.Integer.TYPE) => aliasedParam
127127
}
128128
val expected =
129129
code"""
@@ -153,7 +153,7 @@ class CodeBlockSuite extends SparkFunSuite {
153153
val block = subBlocks.fold(EmptyBlock)(_ + _)
154154
val transformedBlock = block.transform {
155155
case b: Block => b.transformExprValues {
156-
case SimpleExprValue("1 + 1", _) => aliasedParam
156+
case SimpleExprValue("1 + 1", java.lang.Integer.TYPE) => aliasedParam
157157
}
158158
}.asInstanceOf[Blocks]
159159

@@ -182,5 +182,6 @@ class CodeBlockSuite extends SparkFunSuite {
182182
assert(transformedBlock.blocks(1).toString == expected2.toString)
183183
assert(transformedBlock.blocks(2).toString == expected3.toString)
184184
assert(transformedBlock.toString == (expected1 + expected2 + expected3).toString)
185+
assert(transformedBlock.exprValues === Set(isNull, exprInFunc, aliasedParam))
185186
}
186187
}

0 commit comments

Comments
 (0)