8000 feat: Catch and add query context for statement extraction by krlmlr · Pull Request #521 · duckdb/duckdb-r · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat: Catch and add query context for statement extraction #521

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
Merged
merged 1 commit into from
Oct 27, 2024
Merged

feat: Catch and add query context for statement extraction #521

merged 1 commit into from
Oct 27, 2024

Conversation

krlmlr
Copy link
Collaborator
@krlmlr krlmlr commented Oct 26, 2024 •
edited
Loading

Closes tidyverse/duckplyr#219.

library(DBI)

drv <- duckdb::duckdb()
con <- dbConnect(drv)

sql <- "SELECT count(*), date FROM SELECT CAST(time AS date), count(*) GROUP by date;"
dbGetQuery(con, sql)
#> Error: rapi_prepare: Failed to extract statements:
#> Parser Error: syntax error at or near "SELECT"
#> LINE 1: SELECT count(*), date FROM SELECT CAST(time AS date), count(*) GRO...
#>                                    ^

Created on 2024-10-26 with reprex v2.1.1

@krlmlr krlmlr enabled auto-merge October 26, 2024 18:33
@krlmlr krlmlr requested a review from hannes October 26, 2024 18:35
@krlmlr krlmlr mentioned this pull request Oct 26, 2024
Merged
@krlmlr krlmlr added this pull request to the merge queue Oct 26, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Oct 26, 2024
@krlmlr krlmlr added this pull request to the merge queue Oct 27, 2024
Merged via the queue into main with commit 31a5b17 Oct 27, 2024
11 checks passed
@krlmlr krlmlr deleted the f-split-msg branch October 27, 2024 06:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hannes hannes Awaiting requested review from hannes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Better parsing errors
1 participant
@krlmlr
0