pabloem commented on code in PR #23737:
URL: https://github.com/apache/beam/pull/23737#discussion_r999940923


##########
website/www/check-links.sh:
##########
@@ -73,7 +73,7 @@ for external_link in "${external_links[@]}"
 do
     redraw_progress_bar 50 1 $i ${#external_links[@]}
 
-    if ! curl -sSfL --max-time 10 --connect-timeout 10 --retry 2 -4 
"${external_link}" > /dev/null ; then
+    if ! curl -sSfL --max-filesize 2000000 --max-time 10 --connect-timeout 10 
--retry 2 -4 "${external_link}" > /dev/null ; then

Review Comment:
   correct. curl errors out in that scenario, which causes the files to be 
added to the list of troublemakers - but I think this may be fine. I can add 
some printed output and comments stating this.



-- 
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