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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
