cloud-fan commented on code in PR #41932:
URL: https://github.com/apache/spark/pull/41932#discussion_r1266123243
##########
connector/connect/client/jvm/src/main/scala/org/apache/spark/sql/functions.scala:
##########
@@ -7926,12 +7926,13 @@ object functions {
* Call a builtin or temp function.
*
* @param funcName
- * function name
+ * function name that can be qualified using the SQL syntax
* @param cols
* the expression parameters of function
* @since 3.5.0
*/
@scala.annotation.varargs
- def call_function(funcName: String, cols: Column*): Column =
Column.fn(funcName, cols: _*)
+ def call_function(funcName: String, cols: Column*): Column =
+ Column.fn("call_function", lit(funcName) +: cols: _*)
Review Comment:
shall we add a new proto message for it? Currently it may conflict with
calling a temp function named `call_function`.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
