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
