search

evidence-dev / sqltools-duckdb-driver

DuckDB Driver for SQLTools
https://marketplace.visualstudio.com/items?itemName=Evidence.sqltools-duckdb-driver
MIT License
22 stars 7 forks source link

Better Support for Schema / Attached Databases #18

Open aersam opened 4 months ago

aersam commented 4 months ago

Hi there

expanding a table in the tree on the left which is not in the default schema fails: image

image

Attached databases also seem to suffer from this behavior

Otherwise I really like the extension, it's great to work with it

tyw214 commented 4 months ago

actually i have this exact same issue. it doesn't seem to work when there is multiple schema inside of one db... it seems like something to do with not knowing the fully qualified name of the actual table. otherwise, this is a solid schema.

archiewood commented 2 months ago

My assumption here is there is an issue with the generated query for non default schemas, and in the case that the table is in a non-default schema the query needs to be prefixed with the schema:

https://github.com/evidence-dev/sqltools-duckdb-driver/blob/master/src/ls/queries.ts