New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

update bit64::as.integer64 use in test #1567

Open

edward-burn wants to merge 5 commits into tidyverse:main from oxford-pharmacoepi:main

Contributor

edward-burn commented Dec 29, 2024 •

edited

Loading

fixes #1564

@simonpcouch @hadley the check is failing for other reasons I believe - it seems that the error "vector memory limit of 16.0 Gb reached, see mem.maxVSize()" on macos came between https://github.com/tidyverse/dbplyr/actions/runs/11802007517/job/32876673073 and https://github.com/tidyverse/dbplyr/actions/runs/12089965184/job/33716160658

edward-burn added 5 commits

December 29, 2024 12:26


          update bit64::as.integer64 use in test

84ffd2e


          update snaps

aac1108


          update snap

dbcdd7e


          update mysql date test

efd88c0


          undo

43dc424

simonpcouch approved these changes

View reviewed changes

Collaborator

simonpcouch left a comment

Ha, nice catch—just a bad vector recycle. Thumbs up from me, though I'll defer to Hadley on merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet