Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/22403#discussion_r217171729 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/BooleanSimplificationSuite.scala --- @@ -37,6 +38,7 @@ class BooleanSimplificationSuite extends PlanTest with PredicateHelper { Batch("Constant Folding", FixedPoint(50), NullPropagation(conf), ConstantFolding, + SimplifyConditionals, BooleanSimplification, PruneFilters(conf)) :: Nil } --- End diff -- At line 46, SPARK-17851 changed like the following. We need that. ```scala - val testRelation = LocalRelation('a.int, 'b.int, 'c.int, 'd.string) + val testRelation = LocalRelation('a.int, 'b.int, 'c.int, 'd.string, 'e.boolean, 'f.boolean, 'g.boolean, 'h.boolean) ```
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org