Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[doc] Modify the documentation of functions of array_distinct, array_enumerate,array_enumerate_uniq, array_except, array_exists,array_intersect, array_join #1864

Merged
merged 6 commits into from
Jan 23, 2025

Conversation

ixzc
Copy link
Contributor

@ixzc ixzc commented Jan 18, 2025

Modify the documentation of functions of array_distinct, array_enumerate,array_enumerate_uniq, array_except, array_exists,array_intersect, array_join

Versions

  • dev
  • 3.0
  • 2.1
  • 2.0

Languages

  • Chinese
  • English

Docs Checklist

  • Checked by AI
  • Test Cases Built

…enumerate,array_enumerate_uniq, array_except, array_exists,array_intersect, array_join
## Parameters
| Parameter | Description |
|---|---|
| `<arr>` | ARRAY |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@KassieZ KassieZ added has comments Some comments not resolved yet and removed request review labels Jan 20, 2025
@ixzc ixzc requested a review from morrySnow January 22, 2025 04:05
@KassieZ KassieZ added request review and removed has comments Some comments not resolved yet labels Jan 22, 2025

### example
## Return Value
Returns an array.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里需要介绍返回值的含义

```sql
BOOLEAN array_exists(lambda, ARRAY<T> arr1, ARRAY<T> arr2, ... )
BOOLEAN array_exists(ARRAY<T> arr)
ARRAY_EXISTS([lambda,] <arr1> [, <arr2> , ...] )
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
ARRAY_EXISTS([lambda,] <arr1> [, <arr2> , ...] )
ARRAY_EXISTS([ <lambda>, ] <arr1> [, <arr2> , ...] )

@ixzc ixzc requested a review from morrySnow January 22, 2025 07:25
## Parameters
| Parameter | Description |
|---|---|
| `lambda` | A lambda expression that can execute valid scalar functions but does not support aggregate functions. |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
| `lambda` | A lambda expression that can execute valid scalar functions but does not support aggregate functions. |
| `<lambda>` | A lambda expression that can execute valid scalar functions but does not support aggregate functions. |

@ixzc ixzc requested a review from morrySnow January 22, 2025 07:33
@KassieZ KassieZ merged commit c57f7c3 into apache:master Jan 23, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants