You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "thisisnic (via GitHub)" <gi...@apache.org> on 2023/09/20 15:38:25 UTC

[GitHub] [arrow] thisisnic opened a new issue, #37804: [R] Tests which use `with_language()` fail on CRAN

thisisnic opened a new issue, #37804:
URL: https://github.com/apache/arrow/issues/37804

   ### Describe the bug, including details regarding any error messages, version, and platform.
   
   Upon submitting 13.0.0.1 to CRAN, we ended up with a failure on Debian for tests which use `with_language()`:
   
   ```
   ══ Failed tests ════════════════════════════════════════════════════════════════
       ── Error ('test-dplyr-filter.R:291:3'): Filtering on a column that doesn't exist errors correctly ──
       Error: object 'not_a_col' not found
       Backtrace:
            ▆
         1. ├─arrow:::with_language(...) at test-dplyr-filter.R:291:2
         2. │ └─base::force(expr) at tests/testthat/helper-arrow.R:50:2
         3. ├─testthat::expect_warning(...) at test-dplyr-filter.R:296:4
         4. │ └─testthat:::expect_condition_matching(...)
         5. │   └─testthat:::quasi_capture(...)
         6. │     ├─testthat (local) .capture(...)
         7. │     │ └─base::withCallingHandlers(...)
         8. │     └─rlang::eval_bare(quo_get_expr(.quo), quo_get_env(.quo))
         9. ├─testthat::expect_error(...)
        10. │ └─testthat:::expect_condition_matching(...)
        11. │   └─testthat:::quasi_capture(...)
        12. │     ├─testthat (local) .capture(...)
        13. │     │ └─base::withCallingHandlers(...)
        14. │     └─rlang::eval_bare(quo_get_expr(.quo), quo_get_env(.quo))
        15. ├─tbl %>% record_batch() %>% filter(not_a_col == 42) %>% ...
        16. ├─dplyr::collect(.)
        17. ├─dplyr::filter(., not_a_col == 42)
        18. └─arrow:::filter.ArrowTabular(., not_a_col == 42)
        19.   └─base::lapply(filts, arrow_eval, arrow_mask(out))
        20.     └─arrow (local) FUN(X[[i]], ...)
        21.       └─base::tryCatch(...)
        22.         └─base (local) tryCatchList(expr, classes, parentenv, handlers)
        23.           └─base (local) tryCatchOne(expr, names, parentenv, handlers[[1L]])
        24.             └─value[[3L]](cond)
       ── Error ('test-dplyr-mutate.R:520:3'): handle bad expressions ─────────────────
       Error: object 'NOTAVAR' not found
       Backtrace:
            ▆
         1. ├─arrow:::with_language(...) at test-dplyr-mutate.R:520:2
         2. │ └─base::force(expr) at tests/testthat/helper-arrow.R:50:2
         3. ├─testthat::expect_warning(...) at test-dplyr-mutate.R:525:4
         4. │ └─testthat:::expect_condition_matching(...)
         5. │   └─testthat:::quasi_capture(...)
         6. │     ├─testthat (local) .capture(...)
         7. │     │ └─base::withCallingHandlers(...)
         8. │     └─rlang::eval_bare(quo_get_expr(.quo), quo_get_env(.quo))
         9. ├─testthat::expect_error(...)
        10. │ └─testthat:::expect_condition_matching(...)
        11. │   └─testthat:::quasi_capture(...)
        12. │     ├─testthat (local) .capture(...)
        13. │     │ └─base::withCallingHandlers(...)
        14. │     └─rlang::eval_bare(quo_get_expr(.quo), quo_get_env(.quo))
        15. ├─Table$create(tbl) %>% mutate(newvar = NOTAVAR + 2)
        16. ├─dplyr::mutate(., newvar = NOTAVAR + 2)
        17. └─arrow:::mutate.ArrowTabular(., newvar = NOTAVAR + 2)
        18.   └─arrow:::arrow_eval(exprs[[i]], mask)
        19.     └─base::tryCatch(...)
        20.       └─base (local) tryCatchList(expr, classes, parentenv, handlers)
        21.         └─base (local) tryCatchOne(expr, names, parentenv, handlers[[1L]])
        22.           └─value[[3L]](cond)
   
       [ FAIL 2 | WARN 8 | SKIP 109 | PASS 6238 ]
       Error: Test failures
       Execution halted
   ```
   
   ### Component(s)
   
   R


-- 
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: issues-unsubscribe@arrow.apache.org.apache.org

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


[GitHub] [arrow] thisisnic commented on issue #37804: [R] Tests which use `with_language()` fail on CRAN

Posted by "thisisnic (via GitHub)" <gi...@apache.org>.
thisisnic commented on issue #37804:
URL: https://github.com/apache/arrow/issues/37804#issuecomment-1728006317

   Potential culprit: https://github.com/wch/r-source/commit/4f99e1349b4e85231d6b8849cd8720a0f71a8024


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

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


[GitHub] [arrow] thisisnic commented on issue #37804: [R] Tests which use `with_language()` fail on CRAN

Posted by "thisisnic (via GitHub)" <gi...@apache.org>.
thisisnic commented on issue #37804:
URL: https://github.com/apache/arrow/issues/37804#issuecomment-1728001226

   Given these tests passed on 15th September on R-devel, but not today, it could be some change to R-devel between then and now.


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

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