-
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-48116][INFRA][FOLLOWUP] Simplify the build with fixing the if condition #46380
Conversation
… and Daily Python CIs" This reverts commit 1904dee.
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.
+1, LGTM. Thank you for fixing all, @HyukjinKwon .
Feel free to merge at any time when this PR is ready ~ |
Merged to master. |
Reverted. I will make a new try with trying within my own fork. I cherry-picked yours and merged it. |
Oh, thank you for the fast recovery. Looking forward to seeing the new try. |
…ogic ### What changes were proposed in this pull request? This PR is another try of #46380 that is a followup of #46367 that simplifies the build and deduplicate them. ### Why are the changes needed? To fix the condition, and make it deduplicated. ### Does this PR introduce _any_ user-facing change? No, dev-only. ### How was this patch tested? Will test in my own fork: https://github.com/HyukjinKwon/spark/actions/runs/8948215777 ### Was this patch authored or co-authored using generative AI tooling? No. Closes #46381 from HyukjinKwon/SPARK-48116-followup2. Authored-by: Hyukjin Kwon <gurwls223@apache.org> Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
…condition ### What changes were proposed in this pull request? This PR is a followup of apache#46367 that simplifies the build and deduplicate them. ### Why are the changes needed? To fix the condition, and make it deduplicated. ### Does this PR introduce _any_ user-facing change? No, dev-only. ### How was this patch tested? Manually. ### Was this patch authored or co-authored using generative AI tooling? No. Closes apache#46380 from HyukjinKwon/SPARK-48116-followup. Authored-by: Hyukjin Kwon <gurwls223@apache.org> Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
…ogic ### What changes were proposed in this pull request? This PR is another try of apache#46380 that is a followup of apache#46367 that simplifies the build and deduplicate them. ### Why are the changes needed? To fix the condition, and make it deduplicated. ### Does this PR introduce _any_ user-facing change? No, dev-only. ### How was this patch tested? Will test in my own fork: https://github.com/HyukjinKwon/spark/actions/runs/8948215777 ### Was this patch authored or co-authored using generative AI tooling? No. Closes apache#46381 from HyukjinKwon/SPARK-48116-followup2. Authored-by: Hyukjin Kwon <gurwls223@apache.org> Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
…condition ### What changes were proposed in this pull request? This PR is a followup of apache#46367 that simplifies the build and deduplicate them. ### Why are the changes needed? To fix the condition, and make it deduplicated. ### Does this PR introduce _any_ user-facing change? No, dev-only. ### How was this patch tested? Manually. ### Was this patch authored or co-authored using generative AI tooling? No. Closes apache#46380 from HyukjinKwon/SPARK-48116-followup. Authored-by: Hyukjin Kwon <gurwls223@apache.org> Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
…ogic ### What changes were proposed in this pull request? This PR is another try of apache#46380 that is a followup of apache#46367 that simplifies the build and deduplicate them. ### Why are the changes needed? To fix the condition, and make it deduplicated. ### Does this PR introduce _any_ user-facing change? No, dev-only. ### How was this patch tested? Will test in my own fork: https://github.com/HyukjinKwon/spark/actions/runs/8948215777 ### Was this patch authored or co-authored using generative AI tooling? No. Closes apache#46381 from HyukjinKwon/SPARK-48116-followup2. Authored-by: Hyukjin Kwon <gurwls223@apache.org> Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
What changes were proposed in this pull request?
This PR is a followup of #46367 that simplifies the build and deduplicate them.
Why are the changes needed?
To fix the condition, and make it deduplicated.
Does this PR introduce any user-facing change?
No, dev-only.
How was this patch tested?
Manually.
Was this patch authored or co-authored using generative AI tooling?
No.