flyrain commented on code in PR #4624:
URL: https://github.com/apache/polaris/pull/4624#discussion_r3365487215


##########
1.4.1/getting-started/binary-distribution.md:
##########
@@ -34,7 +34,7 @@ Use this guide to quickly start running Polaris using the 
pre-built binary distr
 Download and extract the binary distribution:
 
 ```bash
-curl -L 
https://downloads.apache.org/incubator/polaris/1.4.1/polaris-bin-1.4.1.tgz | 
tar xz
+curl -L https://downloads.apache.org/polaris/1.4.1/polaris-bin-1.4.1.tgz | tar 
xz

Review Comment:
   the link after change seems still not valid,  
   ```
   > curl -L https://downloads.apache.org/polaris/1.4.1/polaris-bin-1.4.1.tgz
   <!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">
   <html><head>
   <title>404 Not Found</title>
   </head><body>
   <h1>Not Found</h1>
   <p>The requested URL was not found on this server.</p>
   </body></html>
   ```
   we may use something like this
   ```
   https://dlcdn.apache.org/polaris/1.4.1/polaris-bin-1.4.1.tgz
   ```



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to