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

Does sergeant really need Java? #7

Closed
psychemedia opened this Issue Jun 16, 2017 · 7 comments

Comments

Projects
None yet
3 participants
@psychemedia
Copy link

psychemedia commented Jun 16, 2017

Hi

I'm running Apache Drill in a docker container linked to a rocker/tidyverse container, which doesn't include Java.

Is there any reason why you use RJDBC rather than DBI?

@hrbrmstr

This comment has been minimized.

Copy link
Owner

hrbrmstr commented Jun 16, 2017

@cgivre

This comment has been minimized.

Copy link

cgivre commented Jun 16, 2017

@hrbrmstr

This comment has been minimized.

Copy link
Owner

hrbrmstr commented Jun 16, 2017

give 0.5.0 a go. I think I managed to successfully auto-decouple rJava & RJDBC but I don't have time to test in a docker container at the moment.

@psychemedia

This comment has been minimized.

Copy link

psychemedia commented Jun 16, 2017

Installing w/ devtools::install_github("hrbrmstr/sergeant") gives:

...
** preparing package for lazy loading
Error in loadNamespace(i, c(lib.loc, .libPaths()), versionCheck = vI[[i]]) : 
  there is no package called ‘RJDBC’
ERROR: lazy loading failed for package ‘sergeant’
* removing ‘/usr/local/lib/R/site-library/sergeant’
Installation failed: Command failed (1)
@hrbrmstr

This comment has been minimized.

Copy link
Owner

hrbrmstr commented Jun 16, 2017

argh. thx. will get docker env setup here for it.

@hrbrmstr

This comment has been minimized.

Copy link
Owner

hrbrmstr commented Jun 16, 2017

think i got it (local rocker test worked with and without rJava/RJDBC)

@psychemedia

This comment has been minimized.

Copy link

psychemedia commented Jun 17, 2017

Lovely... works for me too. Thanks:-)

@hrbrmstr hrbrmstr closed this Jun 17, 2017

hrbrmstr added a commit that referenced this issue Dec 31, 2018

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