Tom Lane wrote:
> Simon Riggs <si...@2ndquadrant.com> writes:
> > On Mon, 2010-02-01 at 20:11 -0300, Alvaro Herrera wrote:
> >> It'd probably be worth changing the order of the ApplySetting calls so
> >> that it doesn't look suspicious.
> 
> > Just a comment would be enough I think
> 
> Yeah.  Changing the order would mean that we'd do extra work applying
> and then removing conflicting settings.  But the general principle here
> is that GUC settings coming from different places are resolved by
> source priority, not order of execution.

C comment patch attached and applied.

-- 
  Bruce Momjian  <br...@momjian.us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +
Index: src/backend/utils/init/postinit.c
===================================================================
RCS file: /cvsroot/pgsql/src/backend/utils/init/postinit.c,v
retrieving revision 1.201
diff -c -c -r1.201 postinit.c
*** src/backend/utils/init/postinit.c	15 Jan 2010 09:19:04 -0000	1.201
--- src/backend/utils/init/postinit.c	5 Feb 2010 20:25:38 -0000
***************
*** 855,860 ****
--- 855,861 ----
  
  	relsetting = heap_open(DbRoleSettingRelationId, AccessShareLock);
  
+ 	/* Later settings are ignored if set earlier. */
  	ApplySetting(databaseid, roleid, relsetting, PGC_S_DATABASE_USER);
  	ApplySetting(InvalidOid, roleid, relsetting, PGC_S_USER);
  	ApplySetting(databaseid, InvalidOid, relsetting, PGC_S_DATABASE);
-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to