[SPARK-28481][SQL] More expressions should extend NullIntolerant

[wangyum]

What changes were proposed in this pull request?

1. Make more expressions extend NullIntolerant.
2. Add a checker(in ExpressionInfoSuite) to identify whether the expression is NullIntolerant.

Why are the changes needed?

Avoid skew join if the join column has many null values and can improve query performance. For examples:

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:

== 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.

[SparkQA]

Test build #123041 has finished for PR 28626 at commit c3625ca.

• This patch fails due to an unknown error code, -9.
• This patch merges cleanly.
• This patch adds no public classes.

[wangyum]

retest this please.

[maropu]

cc: @JoshRosen

[SparkQA]

Test build #123053 has finished for PR 28626 at commit c3625ca.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #123059 has finished for PR 28626 at commit 7fe3490.

• This patch fails Spark unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[maropu]

also cc: @viirya

[SparkQA]

Test build #123105 has finished for PR 28626 at commit b127c41.

• This patch fails due to an unknown error code, -9.
• This patch merges cleanly.
• This patch adds no public classes.

[viirya]

retest this please

[SparkQA]

Test build #123113 has finished for PR 28626 at commit b127c41.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #123124 has finished for PR 28626 at commit 82e97e3.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[wangyum]

@gatorsmile @cloud-fan

[cloud-fan]

ah I see the difference now. Previously we can skip evaluating input1 if input2 is 0. Can we change it back and add comment to explain it? sorry for the back and forth!

[maropu]

Looks okay. Thanks for the updates.

[SparkQA]

Test build #123233 has finished for PR 28626 at commit 1bdff95.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #123237 has finished for PR 28626 at commit 265abd3.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[cloud-fan]

thanks, merging to master!