[SPARK-30339][SQL] Avoid to fail twice in function lookup

[wzhfy]

What changes were proposed in this pull request?

Currently if function lookup fails, spark will give it a second chance by casting decimal type to double type. But for cases where decimal type doesn't exist, it's meaningless to lookup again and cause extra cost like unnecessary metastore access. We should throw exceptions directly in these cases.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Covered by existing tests.

[wzhfy]

cc @dongjoon-hyun @gatorsmile

[SparkQA]

Test build #115679 has finished for PR 26994 at commit a471a9b.

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

[maropu]

retest this please

[SparkQA]

Test build #115688 has finished for PR 26994 at commit a471a9b.

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

[maropu]

retest this please

[SparkQA]

Test build #115753 has finished for PR 26994 at commit a471a9b.

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

[HyukjinKwon]

Can you remove

spark/sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveUDFSuite.scala

Lines 152 to 157 in 9a3dcca

	test("SPARK-16228 Percentile needs explicit cast to double") {
	sql("select percentile(value, cast(0.5 as double)) from values 1,2,3 T(value)")
	sql("select percentile_approx(value, cast(0.5 as double)) from values 1.0,2.0,3.0 T(value)")
	sql("select percentile(value, 0.5) from values 1,2,3 T(value)")
	sql("select percentile_approx(value, 0.5) from values 1.0,2.0,3.0 T(value)")
	}

this test case while we're here? They don't verify this change anymore since Spark implements them now.

[HyukjinKwon]

Looks fine with @maropu's suggestion.

[SparkQA]

Test build #115955 has finished for PR 26994 at commit be6c276.

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

[SparkQA]

Test build #115957 has finished for PR 26994 at commit 71b85e6.

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

[HyukjinKwon]

Merged to master.

[maropu]

Late LGTM. Thanks, @HyukjinKwon !