Skip to content

Commit

Permalink
[SPARK-38321][SQL][TESTS] Fix BooleanSimplificationSuite under ANSI
Browse files Browse the repository at this point in the history
### What changes were proposed in this pull request?
This PR fixes BooleanSimplificationSuite under ANSI mode. Change the test case to be a long type to avoid casting difference in ANSI on/off.

### Why are the changes needed?
To set up a new GA job to run tests with ANSI mode before 3.3.0 release.

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?
Test locally with both ANSI on and off, both passed.

Closes #35654 from anchovYu/ansi-tests-boolean-simplification.

Authored-by: Xinyi Yu <xinyi.yu@databricks.com>
Signed-off-by: Gengliang Wang <gengliang@apache.org>
  • Loading branch information
anchovYu authored and gengliangwang committed Feb 28, 2022
1 parent 309c65a commit 244716f
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -138,8 +138,8 @@ class BooleanSimplificationSuite extends PlanTest with ExpressionEvalHelper with
'a > 1 && 'b > 3 && 'c > 1)

checkCondition(
('a > 1 || 'b > 3) && (('a > 1 || 'b > 3) && 'd > 0 && (('a > 1 || 'b > 3) && 'c > 1)),
('a > 1 || 'b > 3) && 'd > 0 && 'c > 1)
('a > 1 || 'b > 3) && (('a > 1 || 'b > 3) && 'd > 0L && (('a > 1 || 'b > 3) && 'c > 1)),
('a > 1 || 'b > 3) && 'd > 0L && 'c > 1)

checkCondition(
'a > 1 && 'b > 2 && 'a > 1 && 'c > 3,
Expand Down

0 comments on commit 244716f

Please sign in to comment.