dongjoon-hyun commented on a change in pull request #24773:
[SPARK-24544][SQL][FOLLOWUP] Remove a wrong warning on Hive fallback lookup
URL: https://github.com/apache/spark/pull/24773#discussion_r289696377
##########
File path:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveSessionCatalog.scala
##########
@@ -142,8 +142,6 @@ private[sql] class HiveSessionCatalog(
// let's try to load it as a Hive's built-in function.
// Hive is case insensitive.
val functionName = funcName.unquotedString.toLowerCase(Locale.ROOT)
- logWarning("Encountered a failure during looking up function:" +
Review comment:
Yes. Of course, we can add debug messages any place. But, this is a natural
flow for Hive build-in function lookup. To showing a message like `Encountered
a failure during looking up function: NoSuchFunctionException` for every Hive
function at the first invocation will not give much values to the
developer/users. Also, it's misleading to the users because Spark will execute
the function without any problem at the same exeuction.
----------------------------------------------------------------
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]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
