-
Notifications
You must be signed in to change notification settings - Fork 28k
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-48200][INFRA] Split build_python.yml
into per-version cron jobs
#46477
Conversation
I realized that new pipeline is an error-prone style. This will choose the alternative simplest one, cc @HyukjinKwon . |
Could you review this PR, @viirya ? |
Looks good to me.
…On Wed, May 8, 2024 at 9:33 AM Dongjoon Hyun ***@***.***> wrote:
Could you review this PR, @viirya <https://github.com/viirya> ?
—
Reply to this email directly, view it on GitHub
<#46477 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAQZ52WTIU42UA7IVHTFRDZBJHWBAVCNFSM6AAAAABHNI7MIOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBQHE3DMMBYHA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Thank you so much, @viirya ! |
For the record, the result looks clean and good to me.
The following will be triggered tomorrow. |
Thank you @dongjoon-hyun |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm
Thank you, @HyukjinKwon ! |
### What changes were proposed in this pull request? This PR aims to split `build_python.yml` into per-version cron jobs. Technically, this includes a revert of SPARK-48149 and choose [the discussed alternative](apache#46407 (comment)). - apache#46407 - apache#46454 ### Why are the changes needed? To recover Python CI successfully in ASF INFRA policy. - https://github.com/apache/spark/actions/workflows/build_python.yml ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Manual review. ### Was this patch authored or co-authored using generative AI tooling? No. Closes apache#46477 from dongjoon-hyun/SPARK-48200. Authored-by: Dongjoon Hyun <dhyun@apple.com> Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
What changes were proposed in this pull request?
This PR aims to split
build_python.yml
into per-version cron jobs.Technically, this includes a revert of SPARK-48149 and choose the discussed alternative.
build_python.yml
to run a single Python version per cron schedule #46407Why are the changes needed?
To recover Python CI successfully in ASF INFRA policy.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Manual review.
Was this patch authored or co-authored using generative AI tooling?
No.