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