Ngone51 commented on a change in pull request #28979:
URL: https://github.com/apache/spark/pull/28979#discussion_r450269700
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ScalaUDF.scala
##########
@@ -36,6 +36,8 @@ import org.apache.spark.sql.types.{AbstractDataType,
AnyDataType, DataType, User
* @param inputEncoders ExpressionEncoder for each input parameters. For a
input parameter which
* serialized as struct will use encoder instead of
CatalystTypeConverters to
* convert internal value to Scala value.
+ * @param returnEncoder ExpressionEncoder for the return type of function.
It's only defined when
Review comment:
yea, it's more consistent compares to `inputEncoders `
----------------------------------------------------------------
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]
