jonkeane commented on a change in pull request #11965:
URL: https://github.com/apache/arrow/pull/11965#discussion_r769994989
##########
File path: r/R/arrow-package.R
##########
@@ -153,6 +153,9 @@ arrow_with_dataset <- function() {
#' @rdname arrow_available
#' @export
arrow_with_parquet <- function() {
+ # https://github.com/google/snappy/pull/148/files
+ skip_on_linux_devel()
Review comment:
Right, right. I'll pull it out into a separate helper (the renaming of
the skip makes it more accurate with what it does though so I'm inclined to
keep that)
And, yeah we don't need to gate all of parquet, though anywhere we don't set
a default will use snappy. I'm not 100% sure why [only the
example](https://dev.azure.com/ursacomputing/crossbow/_build/results?buildId=16490&view=logs&j=0da5d1d9-276d-5173-c4c4-9d4d4ed14fdb&t=d9b15392-e4ce-5e4c-0c8c-b69645229181&l=12300)
runs into the UBSan, possibly the shape of data / how we are projecting is
unique 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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
