jonkeane commented on a change in pull request #11404:
URL: https://github.com/apache/arrow/pull/11404#discussion_r728954911
##########
File path: r/tests/testthat/test-Array.R
##########
@@ -729,6 +729,9 @@ test_that("Handling string data with embedded nuls", {
)
array_with_nul <- Array$create(raws)$cast(utf8())
+ # test below use/rely on altrep
+ skip_if_r_version("3.5.0")
Review comment:
Yeah, I should probably bring over the comments from the others. We
definitely could conditionally test where the error happens, but since this
would only impact 3.5.0 and below, I think we're ok here. We could make a
follow up for after the release if we want to expand those / make sure we have
coverage on older versions.
--
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]
