Skip to content

Conversation

Y--
Copy link
Collaborator

@Y-- Y-- commented Apr 15, 2025

Add support for 3 new DuckDB options:

  • duckdb.extension_directory - Set the directory to store extensions in, by default in <DataDir>/pg_duckdb/extensions
  • duckdb.temporary_directory - Set the directory to which to write temp files, by default in <DataDir>/pg_duckdb/temp
  • duckdb.max_temp_directory_size - The maximum amount of data stored inside the 'temp_directory' (when set) (e.g., 1GB)

(Cf. DuckDB's documentation https://duckdb.org/docs/stable/configuration/overview)

Solves #666
Fixes #684

@Y-- Y-- requested a review from JelteF April 15, 2025 11:33
@Y-- Y-- changed the title Use C++ construct for connection string Add DuckDB options to GUC Apr 15, 2025
@Y-- Y-- enabled auto-merge (squash) April 15, 2025 11:34
@JelteF JelteF changed the title Add DuckDB options to GUC Add DuckDB temp & extension directory GUCs Apr 15, 2025
@JelteF JelteF disabled auto-merge April 15, 2025 11:35
@JelteF JelteF enabled auto-merge (squash) April 15, 2025 11:35
@Y-- Y-- force-pushed the yl/temp-dir-path branch from da3412e to 70a541a Compare April 15, 2025 13:21
@Y-- Y-- requested a review from JelteF April 15, 2025 13:22
@JelteF JelteF merged commit fce6e4f into main Apr 15, 2025
6 checks passed
@JelteF JelteF deleted the yl/temp-dir-path branch April 15, 2025 14:47
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.

Temporary files should go in the postgres temp directory by default
2 participants