Skip to content

[SPARK-45253][SQL][DOCS] Correct the group of ShiftLeft and ArraySize#43033

Closed
zhengruifeng wants to merge 1 commit intoapache:masterfrom
zhengruifeng:sql_change_group
Closed

[SPARK-45253][SQL][DOCS] Correct the group of ShiftLeft and ArraySize#43033
zhengruifeng wants to merge 1 commit intoapache:masterfrom
zhengruifeng:sql_change_group

Conversation

@zhengruifeng
Copy link
Contributor

What changes were proposed in this pull request?

Correct the group of ShiftLeft and ArraySize

Why are the changes needed?

ShiftLeft should be in the same group of ShiftRight and ShiftRightUnsigned;

ArraySize should be in the same group of ArrayMax, ArrayCompact, etc.

Does this PR introduce any user-facing change?

yes

the two functions will be shown in correct category in https://spark.apache.org/docs/latest/sql-ref-functions.html

How was this patch tested?

manually check

Was this patch authored or co-authored using generative AI tooling?

no

@github-actions github-actions bot added the SQL label Sep 21, 2023
@zhengruifeng zhengruifeng changed the title [SPARK-45253][DOCS] Correct the group of ShiftLeft and ArraySize [SPARK-45253][SQL][DOCS] Correct the group of ShiftLeft and ArraySize Sep 21, 2023
@zhengruifeng
Copy link
Contributor Author

also cc @cloud-fan @HyukjinKwon

@HyukjinKwon
Copy link
Member

Merged to master.

""",
since = "1.5.0",
group = "math_funcs")
group = "bitwise_funcs")
Copy link
Member

Choose a reason for hiding this comment

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

Actually we'll need to update Scaladoc, and Python docs too. functions.scala and funcions.py and functions.R

Copy link
Contributor Author

Choose a reason for hiding this comment

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

will do for python functions, in a separate PR

""",
since = "3.3.0",
group = "collection_funcs")
group = "array_funcs")
Copy link
Member

Choose a reason for hiding this comment

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

this too

Copy link
Contributor Author

Choose a reason for hiding this comment

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

in python, there are many functions in the incorrect groups

I will resolve it in a separate PR

Copy link
Member

Choose a reason for hiding this comment

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

🆗

@zhengruifeng zhengruifeng deleted the sql_change_group branch September 22, 2023 03:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants