[GitHub] [arrow] paleolimbot commented on pull request #12323: ARROW-9235: [R] Support for `connection` class when reading and writing files

[GitBox <gi...@apache.org>]

paleolimbot commented on pull request #12323:
URL: https://github.com/apache/arrow/pull/12323#issuecomment-1031882761


   The Parquet error was, fortunately, not a concurrency issue, but an assumption that the input would be a `RandomAccessFile` (the tests indicate that this was also a problem for the `FeatherReader` in the past). I've added a test so that this case fails in the same way:
   
   ``` r
   # remotes::install_github("paleolimbot/arrow/r@r-connections")
   library(arrow, warn.conflicts = FALSE)
   addr <- "https://github.com/apache/arrow/raw/master/r/inst/v0.7.1.parquet"
   read_parquet(addr)
   #> Error: file must be a "RandomAccessFile"
   ```
   
   I've also removed references to the R_ext/Connections.h header that was causing the CMD check issue...no need to poke that bear yet.
   
   Tomorrow I'll implement `RandomAccessFile`, which should theoretically be possible for `file()` connections and check the other readers for segfaults/threading issues.


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