Skip to content

[AURON #1314] Speed up CI by using Apache closer.lua syntax for dependency downloads#1315

Merged
merrily01 merged 1 commit intoapache:masterfrom
merrily01:speed_up_ci
Oct 13, 2025
Merged

[AURON #1314] Speed up CI by using Apache closer.lua syntax for dependency downloads#1315
merrily01 merged 1 commit intoapache:masterfrom
merrily01:speed_up_ci

Conversation

@merrily01
Copy link
Member

@merrily01 merrily01 commented Sep 18, 2025

Which issue does this PR close?

Closes #1314.

Rationale for this change

Currently, the CI downloads Spark, Celeborn, Uniffle, and other dependencies directly from archive.apache.org, which is relatively slow and sometimes prone to network issues or download failures.

This change is inspired by apache/kyuubi#7205 and @cxzl25 .

Ref : https://infra.apache.org/release-download-pages.html#download-scripts

This PR proposes to:

  • Use APACHE_MIRROR=https://www.apache.org/dyn/closer.lua with ?action=download query to leverage Apache’s mirror network for faster downloads.

  • Adjust artifact extraction (tarball filename handling) to match the mirrored download URLs.

What changes are included in this PR?

Are there any user-facing changes?

How was this patch tested?

@github-actions github-actions bot added the infra label Sep 18, 2025
@merrily01
Copy link
Member Author

merrily01 commented Sep 18, 2025

Depends on #1310 , will rebase afterwards.

@merrily01 merrily01 marked this pull request as ready for review September 24, 2025 09:07
@merrily01
Copy link
Member Author

Rebased and CI passed.

Ready for review @richox @cxzl25

Copy link
Contributor

@ShreyeshArangath ShreyeshArangath left a comment

Choose a reason for hiding this comment

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

Thanks for working on this!

@merrily01
Copy link
Member Author

I noticed that this PR has been open for a while.
Just wanted to check if anyone has remaining concerns or feedback before it gets merged.
Thanks! @richox @cxzl25

@merrily01
Copy link
Member Author

Since there hasn’t been any feedback, I’ll go ahead and merge this. Thanks for the review @cxzl25 @ShreyeshArangath

@merrily01 merrily01 merged commit 47de780 into apache:master Oct 13, 2025
84 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Speed up CI by using Apache closer.lua syntax for dependency downloads

3 participants