[R] Add tests for packages that might be tricky to roundtrip data to Tables + Parquet files #43633
Comments
jonkeane
added a commit
that referenced
this issue
Aug 16, 2024
…p data to Tables + Parquet files (#43634) ### Rationale for this change Add coverage for objects that might have issues roundtripping to Arrow Tables or Parquet files ### What changes are included in this PR? A new test file + a crossbow job that ensures these other packages are installed so the tests run. ### Are these changes tested? The changes are tests ### Are there any user-facing changes? No * GitHub Issue: #43633 Authored-by: Jonathan Keane <[email protected]> Signed-off-by: Jonathan Keane <[email protected]>
|
Issue resolved by pull request 43634 |
jonkeane
added a commit
that referenced
this issue
Aug 17, 2024
Oops, I should have caught this in #43633 Removes `data.table::` since the namespace is loaded. Also fix some linting errors and free up space on the force tests run. Authored-by: Jonathan Keane <[email protected]> Signed-off-by: Jonathan Keane <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the enhancement requested
We recently noticed a problem after revdeps uncovered a lazily read csv with
readrfailed to roundtrip to parquet. Write tests that will be run in our CI (and setup a job that does this, after installing the extra packages) but that will also not trigger complains in checks or on CRAN since we don't need to or want to actually depend on those packages directly.Component(s)
R
The text was updated successfully, but these errors were encountered: