tidyverse / dbplyr

Database (DBI) backend for dplyr

Home Page:https://dbplyr.tidyverse.org

Repository from Github https://github.comtidyverse/dbplyrRepository from Github https://github.comtidyverse/dbplyr

Should name= in compute.tbl_sql() use I() like in copy_to()?

MichaelChirico opened this issue · comments

#' @param name Table name in remote database.

I noticed the new (#1456) recommendation to use I("schema.table") to indicate DBI::Id("schema", "table").

Does that also apply to other verbs such as compute()?

It seems so based on some new test failure I encountered when trying to update {dbplyr} that affected both copy_to() and compute() usages.

I see #1516 did explicitly just this in the NEWS.

Is it better to just add this note to ?compute, or should we aim to create a shared documentation of function arguments where I() is preferred & refer to that instead?