Daniel Possenriede

Results 62 issues of Daniel Possenriede

This is an attempt to only allow `Id()` objects as `prefix` argument to `dbListObjects()` as mentioned in https://github.com/r-dbi/RPostgres/pull/372#issuecomment-1003884250. prefix can now be a list with multiple `Id` objects, so querying...

`dbQuoteIdentifier()` currently does not allow for columns when quoting an `Id()` object. library(DBI) con - Session info --------------------------------------------------------------- #> setting value #> version R version 4.0.2 (2020-06-22) #> os Windows...

feature

`dbListObjects()` does not list empty schemata, i.e. a schema without a table. ``` r library(RPostgres) con [1] 0 dbListObjects(con) #> table is_prefix #> 1 schema = information_schema TRUE #> 2...

`dbListTables()` now uses the [`pg_class`](https://www.postgresql.org/docs/current/catalog-pg-class.html)/[`pg_namespace`](https://www.postgresql.org/docs/current/catalog-pg-namespace.html) tables instead of `INFORMATION_SCHEMA.tables` to retrieve the names of remote tables and (materialized) views. In particular `dbListTables()` will now list: - r = ordinary table...

Should Materialized Views be listed by `dbListTables()`? I'd argue they should since they are objects that share characteristics of both Tables and Views, both of which are returned by `dbListTables()`,...

feature

Please consider adding a warning to `axe_*.default()` methods that the package, which created the object (and might contain more specific `axe_*` methods) is not loaded. I ran into this while...

feature

# Brief description of your issue wingetcreate doesn't support multiple nodes from a single installer. The [PostgreSQL.pgAdmin v5.6 installer manifest](https://github.com/microsoft/winget-pkgs/blob/master/manifests/p/PostgreSQL/pgAdmin/5.6/PostgreSQL.pgAdmin.installer.yaml) for example contains two installer nodes with the same URL,...

Issue-Feature

Some minor fixes to the snapshot.R docs: - snapshots are saved as Markdown files, not JSON (right?) - fix paths to snapshot folder - typo

I think a `expect_snapshot_message()` should be added 1. for consistency. There are `expect_snapshot_error()` and `expect_snapshot_warning()` (#1536). 2. because "sometimes you just want to capture the ~~output or errors~~ message in...

As mentioned in #201, `mutate_geocode(output="all")` currently throws an error. `mutate_geocode()` would have to support list-columns in order for this to work. ``` r suppressPackageStartupMessages({ library(magrittr) library(ggmap) }) data.frame( address =...

feature request