srowen commented on a change in pull request #26712: [SPARK-29883][SQL] Improve
error messages when function name is an alias
URL: https://github.com/apache/spark/pull/26712#discussion_r352204441
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/FunctionRegistry.scala
##########
@@ -118,7 +118,7 @@ class SimpleFunctionRegistry extends FunctionRegistry with
Logging {
throw new AnalysisException(s"undefined function $name")
}
}
- func(children)
+ func(children).setFuncName(name.funcName)
Review comment:
This may not make sense, I don't know this code well, but: is it not that
"Exists" needs to customize its nodeName, for example? or does it never exist
as such a node.
----------------------------------------------------------------
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]
