You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by we...@apache.org on 2023/10/09 04:47:17 UTC

[spark] branch master updated: [SPARK-44913][SQL][FOLLOWUP] DS V2 supports push down V2 UDF that has magic method

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 <li...@hotmail.com>
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 <li...@hotmail.com>
    Signed-off-by: Wenchen Fan <we...@databricks.com>
---
 .../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: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org