799a0116a8
### What changes were proposed in this pull request? This PR proposes to add `BooleanType` support to the `UnwrapCastInBinaryComparison` optimizer that is currently supports `NumericType` only. The main idea is to treat `BooleanType` as 1 bit integer so that we can utilize all optimizations already defined in `UnwrapCastInBinaryComparison`. This work is an extension of SPARK-24994 and SPARK-32858 ### Why are the changes needed? Current implementation of Spark without this PR cannot properly optimize the filter for the following case ``` SELECT * FROM t WHERE boolean_field = 2 ``` The above query creates a filter of `cast(boolean_field, int) = 2`. The casting prevents from pushing down the filter. In contrast, this PR creates a `false` filter and returns early as there cannot be such a matching rows anyway (empty results.) ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Passed existing tests ``` build/sbt "catalyst/test" build/sbt "sql/test" ``` Added unit tests ``` build/sbt "catalyst/testOnly *UnwrapCastInBinaryComparisonSuite -- -z SPARK-36607" build/sbt "sql/testOnly *UnwrapCastInComparisonEndToEndSuite -- -z SPARK-36607" ``` Closes #33865 from kazuyukitanimura/SPARK-36607. Authored-by: Kazuyuki Tanimura <ktanimura@apple.com> Signed-off-by: Wenchen Fan <wenchen@databricks.com> |
||
|---|---|---|
| .. | ||
| benchmarks | ||
| src | ||
| pom.xml | ||