Skip to content
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.

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

modifies DuckDB docs to describe sql literals #1849

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tel
Copy link

@tel tel commented Nov 26, 2024

While it's subtly referenced in the docs and in a few issues, it was not immediately clear to me how to use a customized DuckDB instance as the backing for SQL code blocks. It's not difficult (the db.sql method provides the functionality already) but I definitely struggled for a bit before hunting down the source of DuckDBClient and seeing how simple it is.

This PR explicitly describes how to do this. I documented it on the DuckDB documentation since that's the place that most canonically discusses creating customized DuckDB instances.

See #1846 (comment).

Fil added a commit that referenced this pull request Dec 3, 2024
* illustrates how to make sql fenced code blocks from a DuckDBClient isntance
* syncs the sql fenced code blocks with the mosaic db  (thanks, @tel!)
* gives better guidance for arbitrary query creation (with a safer approach for `xxx IN(${array})`)

supersedes #1849
@Fil Fil mentioned this pull request Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant