Skip to content

[SPARK-47933][CONNECT][PYTHON][FOLLOW-UP] Remove pyspark.sql.classic reference in pyspark.ml.stat - #46262

Closed
HyukjinKwon wants to merge 1 commit into
apache:masterfrom
HyukjinKwon:SPARK-47933-followup4
Closed

[SPARK-47933][CONNECT][PYTHON][FOLLOW-UP] Remove pyspark.sql.classic reference in pyspark.ml.stat#46262
HyukjinKwon wants to merge 1 commit into
apache:masterfrom
HyukjinKwon:SPARK-47933-followup4

Conversation

@HyukjinKwon

@HyukjinKwonHyukjinKwon commented Apr 28, 2024

Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

This PR is a followup of #46155 that removes the reference of _to_seq that pyspark-connect package does not have.

Why are the changes needed?

To recover the CI https://github.com/apache/spark/actions/runs/8861971303

Does this PR introduce any user-facing change?

No, the main change has not been released out yet.

How was this patch tested?

Manually tested.

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

No.

@HyukjinKwon

Copy link
Copy Markdown
MemberAuthor

cc @zhengruifeng

@HyukjinKwon

Copy link
Copy Markdown
MemberAuthor

Thanks!

Merged to master.

HyukjinKwon added a commit that referenced this pull request Apr 30, 2024
… classes
### What changes were proposed in this pull request?
This PR proposes to fix `pyspark-connect` build that is broken by #46155. It is similar with the followups such as #46229 and #46262 but make sure the tests pass within this PR.
### Why are the changes needed?
To recover the CI https://github.com/apache/spark/actions/runs/8884003967
### Does this PR introduce _any_ user-facing change?
No, the main change has not been released yet.
### How was this patch tested?
CI in my own fork: https://github.com/HyukjinKwon/spark/actions/runs/8889698252/job/24408487227
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes#46294 from HyukjinKwon/SPARK-48052.
Authored-by: Hyukjin Kwon <gurwls223@apache.org>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
Sign up for freeto 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.

2 participants

@HyukjinKwon@dongjoon-hyun