Re: [PATCH] Introduce unified support for composite GUC options - Mailing list pgsql-hackers

From Tom Lane
Subject Re: [PATCH] Introduce unified support for composite GUC options
Date
Msg-id 3022888.1758603740@sss.pgh.pa.us
Whole thread Raw
In response to Re: [PATCH] Introduce unified support for composite GUC options  ("David G. Johnston" <david.g.johnston@gmail.com>)
List pgsql-hackers
"David G. Johnston" <david.g.johnston@gmail.com> writes:
> As you note - moving runtime checks to "SET" time has value and this patch
> brings that value.  But it is not evident there is enough value to take on
> the added complexity.  There are few to no requests asking for this ability.

If anything, I'd say we have decades of experience showing that early
checking of GUC values creates more problems than it solves.  There
are too many cases where necessary context is not available at the
time of setting the value.  Particularly, CREATE FUNCTION ... SET
and ALTER DATABASE/USER ... SET are problematic for this.

            regards, tom lane



pgsql-hackers by date:

Previous
From: shveta malik
Date:
Subject: Re: Improve pg_sync_replication_slots() to wait for primary to advance
Next
From: Alena Vinter
Date:
Subject: Re: Resetting recovery target parameters in pg_createsubscriber