cloud-fan commented on a change in pull request #29054:
URL: https://github.com/apache/spark/pull/29054#discussion_r496453529
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
##########
@@ -1299,8 +1299,8 @@ class SessionCatalog(
// Check input argument size
if (e.inputTypes.size != input.size) {
- throw new AnalysisException(s"Invalid number of arguments for function
$name. " +
- s"Expected: ${e.inputTypes.size}; Found: ${input.size}")
+ throw new InvalidFunctionArgumentException(s"Invalid number of
arguments for " +
Review comment:
Instead of using a special exception here, maybe it's better to create a
`InvalidUDFClassException` for the else branch, as it's not really a plan
analysis error.
----------------------------------------------------------------
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]
