Skip to content

[SPARK-47933][CONNECT][PYTHON][FOLLOW-UP] Avoid referencing _to_seq in pyspark-connect - #46229

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

[SPARK-47933][CONNECT][PYTHON][FOLLOW-UP] Avoid referencing _to_seq in pyspark-connect#46229
HyukjinKwon wants to merge 1 commit into
apache:masterfrom
HyukjinKwon:SPARK-47933-followuptmp

Conversation

@HyukjinKwon

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/8821919392/job/24218893631

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.

@dongjoon-hyundongjoon-hyun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1, LGTM (Pending CIs)

@dongjoon-hyun

Copy link
Copy Markdown
Member

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