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-45941][PS] Upgrade pandas to version 2.1.3 #43822

Closed
wants to merge 1 commit into from

Conversation

bjornjorgensen
Copy link
Contributor

@bjornjorgensen bjornjorgensen commented Nov 15, 2023

What changes were proposed in this pull request?

Upgrade pandas from 2.1.2 to 2.1.3

Why are the changes needed?

Fixed infinite recursion from operations that return a new object on some DataFrame subclasses (GH 55763)
and Fix read_parquet() and read_feather() for CVE-2023-47248 (GH 55894)

Release notes for 2.1.3

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass GA

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

No.

@bjornjorgensen bjornjorgensen changed the title [SPARK-45941][PS] Update pandas to version 2.1.3 [SPARK-45941][PS] Upgrade pandas to version 2.1.3 Nov 15, 2023
@dongjoon-hyun
Copy link
Member

Thanks. Could you make CI happy, @bjornjorgensen ?

@bjornjorgensen
Copy link
Contributor Author

it seams to be happy
image

are there anything I can do with this then?
image

@dongjoon-hyun
Copy link
Member

You can provide the link here, @bjornjorgensen ~ That would be enough if the CI is running in your side successfully.

@dongjoon-hyun
Copy link
Member

BTW, your image looks like an old one (3 hours ago). This PR is created 18 minutes ago, isn't it?
Screenshot 2023-11-15 at 1 14 02 PM

@bjornjorgensen
Copy link
Contributor Author

https://github.com/bjornjorgensen/spark/actions/runs/6881177899

I pushed this 3 hours ago and waited to all the tests passed before I open this PR.

@dongjoon-hyun
Copy link
Member

Ya, it looks like that.

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

+1, LGTM. Merged to master.

@bjornjorgensen
Copy link
Contributor Author

Thank you @dongjoon-hyun

@bjornjorgensen bjornjorgensen deleted the pandas-2_1_3 branch November 15, 2023 21:38
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