Re: [HACKERS] Logging configuration changes [REVIEW]

2009-09-17 Thread Peter Eisentraut
On ons, 2009-09-16 at 10:54 +0530, Abhijit Menon-Sen wrote: > I can't help but think that it would be nice to report the default value > of a parameter that is reset (i.e. "parameter $x reset to default value > $y"). The first attached patch does this by calling GetConfigOption() > after the value

Re: [HACKERS] Logging configuration changes [REVIEW]

2009-09-15 Thread Abhijit Menon-Sen
At 2009-09-16 01:18:10 -0500, jcasa...@systemguards.com.ec wrote: > > ok, maybe this is not the most brilliant observation but someone has > to say it... keep the same case in the word "parameter" ;) Oops, thanks. Re²vised patch attached. -- ams diff --git a/src/backend/utils/misc/guc-file.l b/sr

Re: [HACKERS] Logging configuration changes [REVIEW]

2009-09-15 Thread Jaime Casanova
On Wed, Sep 16, 2009 at 12:24 AM, Abhijit Menon-Sen wrote: > > LOG:  received SIGHUP, reloading configuration files > LOG:  parameter "log_connections" reset to default value "off" > LOG:  parameter "log_disconnections" reset to default value "off" > LOG:  Parameter "max_connections" cannot be cha

Re: [HACKERS] Logging configuration changes [REVIEW]

2009-09-15 Thread Abhijit Menon-Sen
(This is my review of the small patch Peter posted on 2009-08-29. See http://archives.postgresql.org/message-id/1251495487.18151.12.ca...@vanquo.pezone.net for the original message.) At 2009-08-29 00:38:07 +0300, pete...@gmx.net wrote: > > Found an easy solution; see attached patch. Neat. The pat