You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@polaris.apache.org by "jbonofre (via GitHub)" <gi...@apache.org> on 2025/09/22 08:54:41 UTC

[PR] Release artifacts should use dlcdn.apache.org (signature and checksum mush refer downloads.apache.org) [polaris]

jbonofre opened a new pull request, #2647:
URL: https://github.com/apache/polaris/pull/2647

   (no comment)


-- 
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: issues-unsubscribe@polaris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Release artifacts should use dlcdn.apache.org (signature and checksum mush refer downloads.apache.org) [polaris]

Posted by "dimas-b (via GitHub)" <gi...@apache.org>.
dimas-b commented on code in PR #2647:
URL: https://github.com/apache/polaris/pull/2647#discussion_r2369749375


##########
site/content/downloads/_index.md:
##########
@@ -71,9 +71,9 @@ Apache Polaris 1.0.1-incubating was released on August 16th, 2025. It's a mainte
 ## 1.0.0
 | Artifact                                                                                                                                                                                         | PGP Sig                                                                                                                                                  | SHA-512 |
 |--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|----------------------------------------------------------------------------------------------------------------------------------------------------------|---|
-| [source tar.gz](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/apache-polaris-1.0.0-incubating.tar.gz)                                                              | [.asc](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/apache-polaris-1.0.0-incubating.tar.gz.asc)                                       | [.sha512](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/apache-polaris-1.0.0-incubating.tar.gz.sha512) |
-| [binary tgz](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/polaris-bin-1.0.0-incubating.tgz)                                                                       | [.asc](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/polaris-bin-1.0.0-incubating.tgz.asc)                                             | [.sha512](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/polaris-bin-1.0.0-incubating.tgz.sha512) |
-| [binary zip](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/polaris-bin-1.0.0-incubating.zip)                                                                       | [.asc](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/polaris-bin-1.0.0-incubating.zip.asc)                                             | [.sha512](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/polaris-bin-1.0.0-incubating.zip.sha512) |
+| [source tar.gz](https://dlcdn.apache.org/incubator/polaris/1.0.0-incubating/apache-polaris-1.0.0-incubating.tar.gz)                                                              | [.asc](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/apache-polaris-1.0.0-incubating.tar.gz.asc)                                       | [.sha512](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/apache-polaris-1.0.0-incubating.tar.gz.sha512) |

Review Comment:
   Oops, my mistake... I did missed that this table row has two cells - LGTM :)



-- 
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: issues-unsubscribe@polaris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Release artifacts should use dlcdn.apache.org (signature and checksum mush refer downloads.apache.org) [polaris]

Posted by "dimas-b (via GitHub)" <gi...@apache.org>.
dimas-b commented on code in PR #2647:
URL: https://github.com/apache/polaris/pull/2647#discussion_r2368951654


##########
site/content/downloads/_index.md:
##########
@@ -71,9 +71,9 @@ Apache Polaris 1.0.1-incubating was released on August 16th, 2025. It's a mainte
 ## 1.0.0
 | Artifact                                                                                                                                                                                         | PGP Sig                                                                                                                                                  | SHA-512 |
 |--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|----------------------------------------------------------------------------------------------------------------------------------------------------------|---|
-| [source tar.gz](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/apache-polaris-1.0.0-incubating.tar.gz)                                                              | [.asc](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/apache-polaris-1.0.0-incubating.tar.gz.asc)                                       | [.sha512](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/apache-polaris-1.0.0-incubating.tar.gz.sha512) |
-| [binary tgz](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/polaris-bin-1.0.0-incubating.tgz)                                                                       | [.asc](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/polaris-bin-1.0.0-incubating.tgz.asc)                                             | [.sha512](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/polaris-bin-1.0.0-incubating.tgz.sha512) |
-| [binary zip](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/polaris-bin-1.0.0-incubating.zip)                                                                       | [.asc](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/polaris-bin-1.0.0-incubating.zip.asc)                                             | [.sha512](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/polaris-bin-1.0.0-incubating.zip.sha512) |
+| [source tar.gz](https://dlcdn.apache.org/incubator/polaris/1.0.0-incubating/apache-polaris-1.0.0-incubating.tar.gz)                                                              | [.asc](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/apache-polaris-1.0.0-incubating.tar.gz.asc)                                       | [.sha512](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/apache-polaris-1.0.0-incubating.tar.gz.sha512) |

Review Comment:
   This is a URL to a checksum, but the PR title says it should refer to `downloads.apache.org`? Could you clarify, please?



-- 
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: issues-unsubscribe@polaris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Release artifacts should use dlcdn.apache.org (signature and checksum mush refer downloads.apache.org) [polaris]

Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on PR #2647:
URL: https://github.com/apache/polaris/pull/2647#issuecomment-3322518560

   @flyrain with dlcdn users can automatically download from a mirror close to them. It avoids to "saturate" downloads.apache.org and speed up for users. 


-- 
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: issues-unsubscribe@polaris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Release artifacts should use dlcdn.apache.org (signature and checksum mush refer downloads.apache.org) [polaris]

Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on code in PR #2647:
URL: https://github.com/apache/polaris/pull/2647#discussion_r2369398278


##########
site/content/downloads/_index.md:
##########
@@ -71,9 +71,9 @@ Apache Polaris 1.0.1-incubating was released on August 16th, 2025. It's a mainte
 ## 1.0.0
 | Artifact                                                                                                                                                                                         | PGP Sig                                                                                                                                                  | SHA-512 |
 |--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|----------------------------------------------------------------------------------------------------------------------------------------------------------|---|
-| [source tar.gz](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/apache-polaris-1.0.0-incubating.tar.gz)                                                              | [.asc](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/apache-polaris-1.0.0-incubating.tar.gz.asc)                                       | [.sha512](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/apache-polaris-1.0.0-incubating.tar.gz.sha512) |
-| [binary tgz](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/polaris-bin-1.0.0-incubating.tgz)                                                                       | [.asc](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/polaris-bin-1.0.0-incubating.tgz.asc)                                             | [.sha512](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/polaris-bin-1.0.0-incubating.tgz.sha512) |
-| [binary zip](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/polaris-bin-1.0.0-incubating.zip)                                                                       | [.asc](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/polaris-bin-1.0.0-incubating.zip.asc)                                             | [.sha512](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/polaris-bin-1.0.0-incubating.zip.sha512) |
+| [source tar.gz](https://dlcdn.apache.org/incubator/polaris/1.0.0-incubating/apache-polaris-1.0.0-incubating.tar.gz)                                                              | [.asc](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/apache-polaris-1.0.0-incubating.tar.gz.asc)                                       | [.sha512](https://downloads.apache.org/incubator/polaris/1.0.0-incubating/apache-polaris-1.0.0-incubating.tar.gz.sha512) |

Review Comment:
   `tar.gz`, `tgz`, `zip` artifacts (source and binary distributions) are using `dlcdn`, the `asc`, `sha512` are still using `downloads`.



-- 
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: issues-unsubscribe@polaris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Release artifacts should use dlcdn.apache.org (signature and checksum mush refer downloads.apache.org) [polaris]

Posted by "flyrain (via GitHub)" <gi...@apache.org>.
flyrain commented on PR #2647:
URL: https://github.com/apache/polaris/pull/2647#issuecomment-3321625910

   Thanks @jbonofre. LGTM. Question: is `dlcdn` supposed to be faster and more optimized to different locations. 


-- 
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: issues-unsubscribe@polaris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Release artifacts should use dlcdn.apache.org (signature and checksum mush refer downloads.apache.org) [polaris]

Posted by "flyrain (via GitHub)" <gi...@apache.org>.
flyrain merged PR #2647:
URL: https://github.com/apache/polaris/pull/2647


-- 
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: issues-unsubscribe@polaris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org