nealrichardson commented on a change in pull request #10724:
URL: https://github.com/apache/arrow/pull/10724#discussion_r671527041
##########
File path: r/tests/testthat/test-dplyr.R
##########
@@ -524,7 +524,7 @@ test_that("is.finite(), is.infinite(), is.nan()", {
) %>% collect(),
df
)
- skip("is.nan() evaluates to NA on NA values (ARROW-12850)")
+ # is.nan() evaluates to FALSE on NA_real_ (ARROW-12850)
Review comment:
Should this comment be updated to note that we've worked around this in
R but that perhaps it should be handled in C++ by ARROW-12850?
--
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
