Only allow duckdb.motherduck_postgres_database in postgresql.conf (#476)

This was always the intention, as we currently don't allow switching the database without restarting postgres. But we forgot to set the correct GUC flags.
duckdb · Dec 4, 2024 · 0b400af · 0b400af
1 parent dff3a16
commit 0b400af
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/docs/settings.md b/docs/settings.md
@@ -36,7 +36,7 @@ Which database to enable MotherDuck support in
 
 Default: `"postgres"`
 
-Access: General
+Access: Needs to be in the `postgresql.conf` file and requires a restart
 
 ### `duckdb.motherduck_default_database`
 

diff --git a/src/pgduckdb.cpp b/src/pgduckdb.cpp
@@ -164,7 +164,7 @@ DuckdbInitGUC(void) {
 	                     PGC_POSTMASTER, GUC_SUPERUSER_ONLY);
 
 	DefineCustomVariable("duckdb.motherduck_postgres_database", "Which database to enable MotherDuck support in",
-	                     &duckdb_motherduck_postgres_database);
+	                     &duckdb_motherduck_postgres_database, PGC_POSTMASTER, GUC_SUPERUSER_ONLY);
 
 	DefineCustomVariable("duckdb.motherduck_default_database",
 	                     "Which database in MotherDuck to designate as default (in place of my_db)",