Skip to content

Conversation

@kevinjqliu
Copy link
Contributor

@kevinjqliu kevinjqliu commented Sep 21, 2025

Rationale for this change

This PR changes the Dockerfile url to use downloads.apache.org/spark first and then fallbacks to https://archive.apache.org/dist.
This should give us speed and reliability.

https://archive.apache.org/dist is very slow, we switch to it because its more reliable and contains all versions of Spark.
downloads.apache.org/spark hosts the latest versions of spark, its typically faster.

Thanks @mccormickt12 for the great idea!

Are these changes tested?

yes

make test-integration-rebuild && make test-integration

also tested fallback logic

Are there any user-facing changes?

@kevinjqliu kevinjqliu requested a review from Fokko September 21, 2025 19:32
Copy link
Contributor

@Fokko Fokko left a comment

Choose a reason for hiding this comment

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

Nice, this was bugging me as well 👍

@kevinjqliu kevinjqliu merged commit caf02ee into apache:main Sep 21, 2025
10 checks passed
@kevinjqliu kevinjqliu deleted the kevinjqliu/add-spark-mirror branch September 21, 2025 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants