jonkeane commented on a change in pull request #9743:
URL: https://github.com/apache/arrow/pull/9743#discussion_r605862157
##########
File path: r/R/util.R
##########
@@ -45,3 +45,16 @@ is_list_of <- function(object, class) {
}
empty_named_list <- function() structure(list(), .Names = character(0))
+
+read_compressed_error <- function(e) {
+ e <- as.character(e)
+ alg <- regmatches(e, gregexpr("(?<=\')(.*?)(?=\')", e, perl = TRUE))[[1]]
Review comment:
OK, we should look for a bit more than that that. Specifically we should
check that the error is about a codec of some sort, and only if it is then
extract the codec out and follow this path. We want to pass on errors that are
not related to codecs not being built. Look at the following:
```
> arrow::read_feather(system.file("v0.7.1.parquet", package = "arrow"))
Error: Invalid: Not a Feather V1 or Arrow IPC file
```
Also, if I'm looking up the regex you have I see some look ahead / look
behind assertions, can you explain what those are doing in there?
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
