Skip to content
This repository has been archived by the owner on May 22, 2024. It is now read-only.

Avoid attaching RPostgreSQL to connect to database #49

Open
iyerr3 opened this issue Oct 24, 2016 · 0 comments
Open

Avoid attaching RPostgreSQL to connect to database #49

iyerr3 opened this issue Oct 24, 2016 · 0 comments

Comments

@iyerr3
Copy link
Contributor

iyerr3 commented Oct 24, 2016

File R/db_conn-generic.R has line
eval(parse(text = paste("suppressMessages(library(", conn.pkg, "))"))),

where conn.pkg can currently only be RPostgreSQL. This pollutes the namespace by attaching all objects from RPostgreSQL. We should avoid this by using the scope operator :: instead of attaching the objects.
@iyerr3 iyerr3 changed the title Removing attaching of RPostgresql package to avoid polluting namespace Remove attaching of RPostgresql package to avoid polluting namespace Oct 24, 2016
@iyerr3 iyerr3 changed the title Remove attaching of RPostgresql package to avoid polluting namespace Avoid attaching RPostgreSQL to connect to database Oct 24, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@iyerr3