Skip to content

Commit f71698d

Browse files
committed
for better code style
1 parent 9f5defa commit f71698d

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/expressions.scala

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -374,8 +374,8 @@ object BooleanSimplification extends Rule[LogicalPlan] with PredicateHelper {
374374

375375
case Not(Not(e)) => e
376376

377-
case Not(e: IsNull) => IsNotNull(e.child)
378-
case Not(e: IsNotNull) => IsNull(e.child)
377+
case Not(IsNull(e)) => IsNotNull(e)
378+
case Not(IsNotNull(e)) => IsNull(e)
379379
}
380380
}
381381
}

sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/BooleanSimplificationSuite.scala

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -240,8 +240,8 @@ class BooleanSimplificationSuite extends PlanTest with ExpressionEvalHelper with
240240
}
241241

242242
test("simplify NOT(IsNull(x)) and NOT(IsNotNull(x))") {
243-
checkCondition(Not(IsNotNull(UnresolvedAttribute("b"))), IsNull(UnresolvedAttribute("b")))
244-
checkCondition(Not(IsNull(UnresolvedAttribute("b"))), IsNotNull(UnresolvedAttribute("b")))
243+
checkCondition(Not(IsNotNull('b)), IsNull('b))
244+
checkCondition(Not(IsNull('b)), IsNotNull('b))
245245
}
246246

247247
protected def assertEquivalent(e1: Expression, e2: Expression): Unit = {

0 commit comments

Comments
 (0)