[GitHub] [arrow] thisisnic commented on a diff in pull request #34708: GH-33287: [R] Cannot read_parquet on http URL

["thisisnic (via GitHub)" <gi...@apache.org>]

thisisnic commented on code in PR #34708:
URL: https://github.com/apache/arrow/pull/34708#discussion_r1156217014


##########
r/R/io.R:
##########
@@ -239,6 +239,14 @@ make_readable_file <- function(file, mmap = TRUE) {
     path <- sub("/$", "", file$base_path)
     file <- filesystem$OpenInputFile(path)
   } else if (is.string(file)) {
+
+    # if this is a HTTP URL, we need a local copy to pass to FileSystem$from_uri

Review Comment:
   Can you expand a bit more on how this is not right? It solves the problem in the issue without any negative side-effects.  



-- 
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: github-unsubscribe@arrow.apache.org

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