-
Notifications
You must be signed in to change notification settings - Fork 485
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
ORC-1811: Use the recommended closer.lua
URL to download ORC format
#2081
Conversation
cc @williamhyun and @wgtmac |
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
Thanks!
@@ -137,7 +137,7 @@ endfunction() | |||
# ---------------------------------------------------------------------- | |||
# ORC Format | |||
ExternalProject_Add (orc-format_ep | |||
URL "https://dlcdn.apache.org/orc/orc-format-${ORC_FORMAT_VERSION}/orc-format-${ORC_FORMAT_VERSION}.tar.gz" | |||
URL "https://www.apache.org/dyn/closer.lua/orc/orc-format-${ORC_FORMAT_VERSION}/orc-format-${ORC_FORMAT_VERSION}.tar.gz?action=download" | |||
URL "https://archive.apache.org/dist/orc/orc-format-${ORC_FORMAT_VERSION}/orc-format-${ORC_FORMAT_VERSION}.tar.gz" |
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.
We can remove this archive.apache.org URL. The closer.lua
URL redirects to the archive.apache.org URL automatically when the target file doesn't exist in https://dist.apache.org/repos/dist/release/orc/ .
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.
Sure, let me clean up this too.
Both the PR description and content are updated. Thank you, @kou . |
This PR aims to use the ASF-recommended `closer.lua` URL instead of the direct `dlcdn` link or `archive` link. - https://infra.apache.org/release-download-pages.html#download-page > you can generate a direct download link using the following syntax: > http://www.apache.org/dyn/closer.lua/bar/foo/foo-5.5.1.zip?action=download To use the recommended download link in order to stabilize CIs. This is suggested from Arrow community. - #1830 (comment) - apache/arrow#44977 Pass the CIs. No. Closes #2081 from dongjoon-hyun/ORC-1811. Authored-by: Dongjoon Hyun <dongjoon@apache.org> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org> (cherry picked from commit 28fe676) Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
Merged to main/2.0. |
What changes were proposed in this pull request?
This PR aims to use the ASF-recommended
closer.lua
URL instead of the directdlcdn
link orarchive
link.https://infra.apache.org/release-download-pages.html#download-page
Why are the changes needed?
To use the recommended download link in order to stabilize CIs.
This is suggested from Arrow community.
How was this patch tested?
Pass the CIs.
Was this patch authored or co-authored using generative AI tooling?
No.