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

[SPARK-43941][SQL][PYTHON][CONNECT] Add any_value, approx_percentile, count_if, first_value, histogram_numeric, last_value, reduce to Scala, Python and Connect API #41588

Closed
wants to merge 2 commits into from

Conversation

beliefer
Copy link
Contributor

@beliefer beliefer commented Jun 14, 2023

What changes were proposed in this pull request?

This PR want add any_value, approx_percentile, count_if, first_value, histogram_numeric, last_value, reduce to Scala, Python and Connect API.

Why are the changes needed?

Add any_value, approx_percentile, count_if, first_value, histogram_numeric, last_value, reduce to Scala, Python and Connect API

Does this PR introduce any user-facing change?

'No'.
New feature.

How was this patch tested?

New test cases.

…count_if,first_value,histogram_numeric,last_value to Scala and Python
@beliefer
Copy link
Contributor Author

ping @zhengruifeng cc @HyukjinKwon

@zhengruifeng
Copy link
Contributor

merged to master

@beliefer
Copy link
Contributor Author

@zhengruifeng Thank you!

czxm pushed a commit to czxm/spark that referenced this pull request Jun 19, 2023
… count_if, first_value, histogram_numeric, last_value, reduce to Scala, Python and Connect API

### What changes were proposed in this pull request?
This PR want add `any_value`, `approx_percentile`, `count_if`, `first_value`, `histogram_numeric`, `last_value`, `reduce` to Scala, Python and Connect API.

### Why are the changes needed?
Add `any_value`, `approx_percentile`, `count_if`, `first_value`, `histogram_numeric`, `last_value`, `reduce` to Scala, Python and Connect API

### Does this PR introduce _any_ user-facing change?
'No'.
New feature.

### How was this patch tested?
New test cases.

Closes apache#41588 from beliefer/SPARK-43941.

Authored-by: Jiaan Geng <beliefer@163.com>
Signed-off-by: Ruifeng Zheng <ruifengz@apache.org>
HyukjinKwon added a commit that referenced this pull request Aug 22, 2023
…ons.approx_percentile` to 3.5.0

### What changes were proposed in this pull request?
`sql.functions.approx_percentile`  was introduced in SPARK-43941 (#41588). This is a pr for the Spark 3.5.0 and it does not belong to Spark 3.4.0. Therefore, this pr corrects the `versionadded` of `sql.functions.approx_percentile` to 3.5.0 and removed `versionchanged`.

### Why are the changes needed?
Correct the `versionadded` of `sql.functions.approx_percentile`.

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
Pass GitHub Actions

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

Closes #42597 from LuciferYang/SPARK-44904.

Lead-authored-by: yangjie01 <yangjie01@baidu.com>
Co-authored-by: Hyukjin Kwon <gurwls223@gmail.com>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
HyukjinKwon added a commit that referenced this pull request Aug 22, 2023
…ons.approx_percentile` to 3.5.0

### What changes were proposed in this pull request?
`sql.functions.approx_percentile`  was introduced in SPARK-43941 (#41588). This is a pr for the Spark 3.5.0 and it does not belong to Spark 3.4.0. Therefore, this pr corrects the `versionadded` of `sql.functions.approx_percentile` to 3.5.0 and removed `versionchanged`.

### Why are the changes needed?
Correct the `versionadded` of `sql.functions.approx_percentile`.

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
Pass GitHub Actions

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

Closes #42597 from LuciferYang/SPARK-44904.

Lead-authored-by: yangjie01 <yangjie01@baidu.com>
Co-authored-by: Hyukjin Kwon <gurwls223@gmail.com>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
(cherry picked from commit 534f9ea)
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
valentinp17 pushed a commit to valentinp17/spark that referenced this pull request Aug 24, 2023
…ons.approx_percentile` to 3.5.0

### What changes were proposed in this pull request?
`sql.functions.approx_percentile`  was introduced in SPARK-43941 (apache#41588). This is a pr for the Spark 3.5.0 and it does not belong to Spark 3.4.0. Therefore, this pr corrects the `versionadded` of `sql.functions.approx_percentile` to 3.5.0 and removed `versionchanged`.

### Why are the changes needed?
Correct the `versionadded` of `sql.functions.approx_percentile`.

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
Pass GitHub Actions

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

Closes apache#42597 from LuciferYang/SPARK-44904.

Lead-authored-by: yangjie01 <yangjie01@baidu.com>
Co-authored-by: Hyukjin Kwon <gurwls223@gmail.com>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
ragnarok56 pushed a commit to ragnarok56/spark that referenced this pull request Mar 2, 2024
…ons.approx_percentile` to 3.5.0

### What changes were proposed in this pull request?
`sql.functions.approx_percentile`  was introduced in SPARK-43941 (apache#41588). This is a pr for the Spark 3.5.0 and it does not belong to Spark 3.4.0. Therefore, this pr corrects the `versionadded` of `sql.functions.approx_percentile` to 3.5.0 and removed `versionchanged`.

### Why are the changes needed?
Correct the `versionadded` of `sql.functions.approx_percentile`.

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
Pass GitHub Actions

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

Closes apache#42597 from LuciferYang/SPARK-44904.

Lead-authored-by: yangjie01 <yangjie01@baidu.com>
Co-authored-by: Hyukjin Kwon <gurwls223@gmail.com>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
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.

2 participants