This is an automated email from the ASF dual-hosted git repository.

ruifengz 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 644c8b25bfd [SPARK-45335][SQL][DOCS] Correct the group of `ElementAt` 
and `TryElementAt`
644c8b25bfd is described below

commit 644c8b25bfda1fe323a7b621d14a340018560136
Author: Ruifeng Zheng <ruife...@apache.org>
AuthorDate: Wed Sep 27 14:03:31 2023 +0800

    [SPARK-45335][SQL][DOCS] Correct the group of `ElementAt` and `TryElementAt`
    
    ### What changes were proposed in this pull request?
    Correct the group of `ElementAt` and `TryElementAt`, they both support 
array and map input, so should be in `collection functions`.
    
    Existing category strategy seems like this: if a function support more than 
one types, it should be in `collection functions`:
    
    - `Size` supports both array and map, it is in `collection functions`;
    - `Reverse` supports both array and string, it is in `collection functions`;
    
    So far, I didn't find other places with incorrect groups.
    
    ### Why are the changes needed?
    for docs
    
    ### Does this PR introduce _any_ user-facing change?
    yes, they will be in `collection functions` in SQL references
    
    ### How was this patch tested?
    CI
    
    ### Was this patch authored or co-authored using generative AI tooling?
    No
    
    Closes #43121 from zhengruifeng/group_element_at.
    
    Lead-authored-by: Ruifeng Zheng <ruife...@apache.org>
    Co-authored-by: Ruifeng Zheng <ruife...@foxmail.com>
    Signed-off-by: Ruifeng Zheng <ruife...@apache.org>
---
 .../apache/spark/sql/catalyst/expressions/collectionOperations.scala  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
 
b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
index 4a3c7bbc2be..759000bc5f5 100644
--- 
a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
+++ 
b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
@@ -2333,7 +2333,7 @@ case class Get(
        b
   """,
   since = "2.4.0",
-  group = "map_funcs")
+  group = "collection_funcs")
 case class ElementAt(
     left: Expression,
     right: Expression,
@@ -2557,7 +2557,7 @@ case class ElementAt(
        b
   """,
   since = "3.3.0",
-  group = "map_funcs")
+  group = "collection_funcs")
 case class TryElementAt(left: Expression, right: Expression, replacement: 
Expression)
   extends RuntimeReplaceable with InheritAnalysisRules {
   def this(left: Expression, right: Expression) =


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to