Skip to content

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

Merged
krlmlr merged 1 commit intomainfrom
f-split-msg
Oct 27, 2024
Merged

feat: Catch and add query context for statement extraction#521
krlmlr merged 1 commit intomainfrom
f-split-msg

Conversation

@krlmlr
Copy link
Copy Markdown
Collaborator

@krlmlr krlmlr commented Oct 26, 2024

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 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
@krlmlr krlmlr deleted the f-split-msg branch October 27, 2024 06:46
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Oct 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Better parsing errors

1 participant