nealrichardson commented on a change in pull request #11656:
URL: https://github.com/apache/arrow/pull/11656#discussion_r746579482



##########
File path: r/vignettes/dataset.Rmd
##########
@@ -48,7 +48,10 @@ arrow::copy_files("s3://ursa-labs-taxi-data", "nyc-taxi")
 ```
 
 If your arrow build doesn't have S3 support, you can download the files
-with some additional code:
+with the additional code shown below.  Note, that as these are large files, 
+to successfully download them, you may first need to increase your download 
+timeout from the default of 60 seconds, e.g.
+`options(timeout = max(300, getOption("timeout")))`.

Review comment:
       I get that this is a more correct solution, but if we're saying the 
default is 60, why not just:
   
   ```suggestion
   `options(timeout = 300)`.
   ```
   
   (the sentence says "e.g." so this is just an example, not the only absolute 
truth)




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