Skip to content

[SPARK-52972][PYTHON][INFRA] Remove the testing image for python 3.9#51682

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

[SPARK-52972][PYTHON][INFRA] Remove the testing image for python 3.9#51682
zhengruifeng wants to merge 1 commit intoapache:masterfrom
zhengruifeng:del_py_39_image

Conversation

@zhengruifeng
Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Remove the testing image for python 3.9

Why are the changes needed?

Python 3.9 support has been dropped

Does this PR introduce any user-facing change?

No

How was this patch tested?

CI

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

No

Copy link
Copy Markdown
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.

What happens for branch-4.0? I guess it builds and use its own Python 3.9 image via the following, @zhengruifeng ?

- 'dev/spark-test-image/python-309/Dockerfile'

@zhengruifeng
Copy link
Copy Markdown
Contributor Author

What happens for branch-4.0? I guess it builds and use its own Python 3.9 image via the following, @zhengruifeng ?

- 'dev/spark-test-image/python-309/Dockerfile'

branch-4.0 should use its own image, but I am not sure whether this cache workflow is still used in 4.0.

maybe we can just add a comment in the imagefile that it is not used since 4.1?

@zhengruifeng
Copy link
Copy Markdown
Contributor Author

@dongjoon-hyun after some investigation, I think it won't affect 4.0 if we don't plan to upgrade this image for 4.0.
I will keep monitoring the 4.0 CI.

@zhengruifeng
Copy link
Copy Markdown
Contributor Author

thanks all, merged to master

@zhengruifeng zhengruifeng deleted the del_py_39_image branch July 29, 2025 03:22
@dongjoon-hyun
Copy link
Copy Markdown
Member

Sure. Thank you!

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.

4 participants