You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by "dongjoon-hyun (via GitHub)" <gi...@apache.org> on 2024/05/08 16:20:26 UTC
[PR] [SPARK-48200][INFRA] Split `build_python.yml` into per-version cron jobs [spark]
dongjoon-hyun opened a new pull request, #46477:
URL: https://github.com/apache/spark/pull/46477
### What changes were proposed in this pull request?
This PR aims to split `build_python.yml` into per-version cron jobs.
Technically, this is a revert of SPARK-48149 .
### Why are the changes needed?
### Does this PR introduce _any_ user-facing change?
### How was this patch tested?
Manual review.
### Was this patch authored or co-authored using generative AI tooling?
<!--
If generative AI tooling has been used in the process of authoring this patch, please include the
phrase: 'Generated-by: ' followed by the name of the tool and its version.
If no, write 'No'.
Please refer to the [ASF Generative Tooling Guidance](https://www.apache.org/legal/generative-tooling.html) for details.
-->
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
Re: [PR] [SPARK-48200][INFRA] Split `build_python.yml` into per-version cron jobs [spark]
Posted by "dongjoon-hyun (via GitHub)" <gi...@apache.org>.
dongjoon-hyun commented on PR #46477:
URL: https://github.com/apache/spark/pull/46477#issuecomment-2101586284
Thank you, @HyukjinKwon !
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
Re: [PR] [SPARK-48200][INFRA] Split `build_python.yml` into per-version cron jobs [spark]
Posted by "dongjoon-hyun (via GitHub)" <gi...@apache.org>.
dongjoon-hyun commented on PR #46477:
URL: https://github.com/apache/spark/pull/46477#issuecomment-2100966088
Could you review this PR, @viirya ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
Re: [PR] [SPARK-48200][INFRA] Split `build_python.yml` into per-version cron jobs [spark]
Posted by "dongjoon-hyun (via GitHub)" <gi...@apache.org>.
dongjoon-hyun commented on PR #46477:
URL: https://github.com/apache/spark/pull/46477#issuecomment-2101263605
For the record, the result looks clean and good to me.
- https://github.com/apache/spark/actions/workflows/build_python_3.10.yml
- https://github.com/apache/spark/actions/workflows/build_python_3.12.yml
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
Re: [PR] [SPARK-48200][INFRA] Split `build_python.yml` into per-version cron jobs [spark]
Posted by "dongjoon-hyun (via GitHub)" <gi...@apache.org>.
dongjoon-hyun closed pull request #46477: [SPARK-48200][INFRA] Split `build_python.yml` into per-version cron jobs
URL: https://github.com/apache/spark/pull/46477
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
Re: [PR] [SPARK-48200][INFRA] Split `build_python.yml` into per-version cron jobs [spark]
Posted by "viirya (via GitHub)" <gi...@apache.org>.
viirya commented on PR #46477:
URL: https://github.com/apache/spark/pull/46477#issuecomment-2101276089
Thank you @dongjoon-hyun
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
Re: [PR] [SPARK-48200][INFRA] Split `build_python.yml` into per-version cron jobs [spark]
Posted by "dongjoon-hyun (via GitHub)" <gi...@apache.org>.
dongjoon-hyun commented on PR #46477:
URL: https://github.com/apache/spark/pull/46477#issuecomment-2100976472
Thank you so much, @viirya !
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
Re: [PR] [SPARK-48200][INFRA] Split `build_python.yml` into per-version cron jobs [spark]
Posted by "dongjoon-hyun (via GitHub)" <gi...@apache.org>.
dongjoon-hyun commented on PR #46477:
URL: https://github.com/apache/spark/pull/46477#issuecomment-2100955572
I realized that new pipeline is an error-prone style. This will choose [the alternative simplest one](https://github.com/apache/spark/pull/46407#discussion_r1591586209), cc @HyukjinKwon .
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org
Re: [PR] [SPARK-48200][INFRA] Split `build_python.yml` into per-version cron jobs [spark]
Posted by "viirya (via GitHub)" <gi...@apache.org>.
viirya commented on PR #46477:
URL: https://github.com/apache/spark/pull/46477#issuecomment-2100974388
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
> <https://github.com/apache/spark/pull/46477#issuecomment-2100966088>, or
> unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AAAQZ52WTIU42UA7IVHTFRDZBJHWBAVCNFSM6AAAAABHNI7MIOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBQHE3DMMBYHA>
> .
> You are receiving this because you were mentioned.Message ID:
> ***@***.***>
>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org