Skip to content

Provide necessary entry points for environment scan in read_duckdb_sql() #645

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.

Sign up for GitHub

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

Jump to bottom
Open
krlmlr opened this issue Dec 16, 2024 · 1 comment · May be fixed by #1077
Open

Provide necessary entry points for environment scan in read_duckdb_sql() #645

krlmlr opened this issue Dec 16, 2024 · 1 comment · May be fixed by #1077
Labels
duckplyr 🗜️ Support for the duckplyr R package

Comments

@krlmlr
Copy link
Collaborator

krlmlr commented Dec 16, 2024 

con <- DBI::dbConnect(duckdb::duckdb(environment_scan = TRUE))
duckplyr::ducksql("FROM mtcars", con = con)
#> Error in `duckplyr::ducksql()`:
#> ! use of NULL environment is defunct
DBI::dbDisconnect(con)

Created on 2024-12-16 with reprex v2.1.1
@krlmlr krlmlr added the duckplyr 🗜️ Support for the duckplyr R package label Feb 12, 2025
@krlmlr
Copy link
Collaborator Author

krlmlr commented Mar 11, 2025 •
edited
Loading

Upstream reprex:

con <- DBI::dbConnect(duckdb::duckdb(environment_scan = TRUE))
duckdb:::rel_from_sql(con, "FROM mtcars")
#> Error:
#> ! use of NULL environment is defunct
DBI::dbDisconnect(con)

Created on 2025-03-11 with reprex v2.1.1

Also:

con <- DBI::dbConnect(duckdb::duckdb(environment_scan = TRUE))
df <- data.frame(a = 1)
duckdb:::rel_from_sql(con, "FROM df")
#> Error:
#> ! use of NULL environment is defunct
DBI::dbDisconnect(con)

Created on 2025-03-11 with reprex v2.1.1

@krlmlr krlmlr changed the title Provide necessary entry points for environment scan in ducksql() Provide necessary entry points for environment scan in read_duckdb_sql() Mar 13, 2025
@krlmlr krlmlr linked a pull request Mar 13, 2025 that will close this issue
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
duckplyr 🗜️ Support for the duckplyr R package
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: New env argument to rel_from_sql() duckdb/duckdb-r
1 participant
@krlmlr