search

duckdb / dbt-duckdb

dbt (http://getdbt.com) adapter for DuckDB (http://duckdb.org)
Apache License 2.0
792 stars 70 forks source link

Re-enable returning a temp table in python models #272

Closed AlexanderVR closed 8 months ago

AlexanderVR commented 8 months ago

The following sequence of MRs added cursor isolation when writing back to Duckdb:

This has the consequence that any dbt python model that returns a DuckDbPyRelation referring to a temporary table will fail, since temp tables do not cross cursor boundaries.

We add a special code path to re-enable this case.

jwills commented 8 months ago

Ack, thank you so much @AlexanderVR ! cc'ing @droher for his awareness as well.