jonkeane commented on code in PR #13014:
URL: https://github.com/apache/arrow/pull/13014#discussion_r860004786


##########
r/tests/testthat/test-parquet.R:
##########
@@ -197,6 +197,8 @@ test_that("Maps are preserved when writing/reading from 
Parquet", {
 })
 
 test_that("read_parquet() and write_parquet() accept connection objects", {
+  skip_if_not_available("snappy")

Review Comment:
   Yeah, I've been thinking about how may of these we need + it would be great 
if we actually tested this with our default and then fell back on sanitizers 
only. Either compression should be totally fine, but it would be nice to have 
coverage on both. But we should do that generally. I'll make a Jira for that + 
we can do that outside of the release pressure. 



-- 
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-unsubscr...@arrow.apache.org

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

Reply via email to