Skip to content
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.

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

ARROW-3795: [R] Support for retrieving NAs from INT64 arrays #2982

Closed

Conversation

romainfrancois
Copy link
Contributor

library(arrow)
#> 
#> Attaching package: 'arrow'
#> The following object is masked from 'package:utils':
#> 
#>     timestamp
#> The following objects are masked from 'package:base':
#> 
#>     array, table

x <- bit64::as.integer64(c(NA))
a <- array(x)
a$as_vector()
#> integer64
#> [1] <NA>

Created on 2018-11-16 by the reprex package (v0.2.1.9000)

@romainfrancois
Copy link
Contributor Author

ping @javierluraschi

Copy link
Member

@wesm wesm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@wesm
Copy link
Member

wesm commented Nov 16, 2018

* DONE (devtools)
The downloaded source packages are in
	‘/tmp/Rtmpe50Hvl/downloaded_packages’
4.32s$ Rscript -e 'deps <- devtools::dev_package_deps(dependencies = NA);devtools::install_deps(dependencies = TRUE);if (!all(deps$package %in% installed.packages())) { message("missing: ", paste(setdiff(deps$package, installed.packages()), collapse=", ")); q(status = 1, save = "no")}'
Error: HTTP error 403.
  API rate limit exceeded for 35.188.73.34. (But here's the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.)
  Rate limit remaining: 0
  Rate limit reset at: 2018-11-16 21:15:57 UTC
Execution halted

It would help give me peace of mind merging patches if I could easily run the unit test suite locally (preferably from the command line so I can add a bash function to my scripts). I opened https://issues.apache.org/jira/browse/ARROW-3824 about documenting this in the README

@wesm wesm closed this in 29657d8 Nov 16, 2018
@romainfrancois
Copy link
Contributor Author

Sure. Will document that. Is it possible to clear the travis cache as @jimhester hinted ? I don't have access so can't do it on the main repo.

image

@wesm
Copy link
Member

wesm commented Nov 16, 2018

Yes I'll try that now; there are a lot of caches!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants