Skip to content

Connect Superset to DuckDB read-only - #4

Closed
cnstlungu wants to merge 2 commits into
mainfrom
superset-readonly
Closed

cnstlungu wants to merge 2 commits into
mainfrom
superset-readonly

Conversation

@cnstlungu

Copy link
Copy Markdown
Owner

Testing whether read-only access removes the uid dependency. The CI chmod workaround is removed in this branch so the run actually exercises the fix.

@cnstlungu

Copy link
Copy Markdown
Owner Author

Closing: disproven. A direct duckdb.connect(read_only=True) inside the container opens the 644 file owned by another uid just fine, but access_mode=read_only never reaches DuckDB — superset import-dashboards defines the connection from the URI embedded in dashboard.zip, overriding what set_database_uri stored, and connects read-write. Superset therefore needs write permission on the warehouse file; the fix has to grant it.

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