Skip to content

Conversation

@zilto
Copy link
Collaborator

@zilto zilto commented Sep 29, 2025

In dlt / sqlalchemy, postgresql or postgres are interchangeable values for drivername. Though, we pass this value to duckdb and it doesn't handle postgresql.

Connection with `client_type=DuckLakeSqlClient` to `dataset_name=repro_dataset_20250929040657` failed. Please check if you configured the credentials at all and provided the right credentials values. You can be also denied access or your internet connection may be down. The actual reason given is: IO Error: Failed to attach DuckLake MetaData "__ducklake_metadata_lake_catalog" at path + "postgresql:postgresql://loader:loader@localhost:5432/dlt_data"Extension "/home/tjean/.duckdb/extensions/v1.3.2/linux_amd64/postgresql.duckdb_extension" not found.

Candidate extensions: "postgres", "postgres_scanner", "sqlite", "sqlite3", "parquet"

Changes

  • Explicitly convert postgresql to postgres when writing the ATTACH statement. It should be the only place where it's relevant
  • Write a unit test

@zilto zilto self-assigned this Sep 29, 2025
@zilto zilto added the bug Something isn't working label Sep 29, 2025
@zilto zilto requested a review from anuunchin September 29, 2025 16:25
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Sep 29, 2025

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
docs 362d2f8 Commit Preview URL

Branch Preview URL
Sep 29 2025, 04:35 PM

Copy link
Contributor

@anuunchin anuunchin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧠

@zilto zilto merged commit 940edd5 into devel Sep 30, 2025
112 of 115 checks passed
@zilto zilto deleted the fix/ducklake-postgresql-driver branch September 30, 2025 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants