This is an automated email from the ASF dual-hosted git repository.
wenchen pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new ffc6994f465 [SPARK-44913][SQL][FOLLOWUP] DS V2 supports push down V2
UDF that has magic method
ffc6994f465 is described below
commit ffc6994f46534dd955a387cea51afb9a67cb781e
Author: Xianyang Liu <[email protected]>
AuthorDate: Mon Oct 9 12:47:00 2023 +0800
[SPARK-44913][SQL][FOLLOWUP] DS V2 supports push down V2 UDF that has magic
method
### What changes were proposed in this pull request?
This is follow up #42612 and to address the comment
https://github.com/apache/spark/pull/42612#discussion_r1345276803
### Why are the changes needed?
To address comments.
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
Existing UTs.
### Was this patch authored or co-authored using generative AI tooling?
No
Closes #43262 from ConeyLiu/42612-followup.
Authored-by: Xianyang Liu <[email protected]>
Signed-off-by: Wenchen Fan <[email protected]>
---
.../org/apache/spark/sql/catalyst/expressions/objects/objects.scala | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
index 4cfc40f4479..a3e049c980c 100644
---
a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
+++
b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
@@ -355,7 +355,7 @@ case class StaticInvoke(
if (scalarFunction.nonEmpty) {
super.stringArgs
} else {
- super.stringArgs.take(8)
+ super.stringArgs.toSeq.dropRight(1).iterator
}
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
