wangyum opened a new pull request #28626:
URL: https://github.com/apache/spark/pull/28626
### What changes were proposed in this pull request?
1. Make more expressions extend `NullIntolerant`.
2. Add a checker(`NullIntolerantCheckerSuite`) to identify whether the
expression is `NullIntolerant`.
### Why are the changes needed?
Improve query performance. For examples:
```sql
CREATE TABLE t1(c1 string, c2 string) USING parquet;
CREATE TABLE t2(c1 string, c2 string) USING parquet;
EXPLAIN SELECT t1.* FROM t1 JOIN t2 ON upper(t1.c1) = upper(t2.c1);
```
Before and after this PR:
```sql
== Physical Plan ==
*(2) Project [c1#5, c2#6]
+- *(2) BroadcastHashJoin [upper(c1#5)], [upper(c1#7)], Inner, BuildLeft
:- BroadcastExchange HashedRelationBroadcastMode(List(upper(input[0,
string, true]))), [id=#41]
: +- *(1) ColumnarToRow
: +- FileScan parquet default.t1[c1#5,c2#6]
+- *(2) ColumnarToRow
+- FileScan parquet default.t2[c1#7]
== Physical Plan ==
*(2) Project [c1#5, c2#6]
+- *(2) BroadcastHashJoin [upper(c1#5)], [upper(c1#7)], Inner, BuildRight
:- *(2) Project [c1#5, c2#6]
: +- *(2) Filter isnotnull(c1#5)
: +- *(2) ColumnarToRow
: +- FileScan parquet default.t1[c1#5,c2#6]
+- BroadcastExchange HashedRelationBroadcastMode(List(upper(input[0,
string, true]))), [id=#59]
+- *(1) Project [c1#7]
+- *(1) Filter isnotnull(c1#7)
+- *(1) ColumnarToRow
+- FileScan parquet default.t2[c1#7]
```
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Unit test.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
