Re: Latch-ifying the syslogger process
Andrew Dunstan <andrew@dunslane.net>
From: Andrew Dunstan <andrew@dunslane.net>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: pgsql-hackers@postgreSQL.org
Date: 2012-05-12T22:59:20Z
Lists: pgsql-hackers
On 05/12/2012 04:00 PM, Andrew Dunstan wrote: > > > On 05/12/2012 03:36 PM, Tom Lane wrote: >> I noticed a large oversight in our efforts to reduce the server's idle >> wakeup frequency: if you've got logging_collector turned on, the >> syslogger process will wake up once a second, whether it has anything >> to do or not. But the only reasons it has for waking up are signals, >> data arrival, and time-based logfile rotation, and it is easy to >> calculate the time until the next logfile rotation event. So this >> seems really easy to latch-ify, and I would like to apply the attached >> patch if there are not objections. I do not however have the ability >> to test the Windows side of it, so it'd be nice if someone would check >> that that still works (particularly, that it shuts down cleanly). > > > I can do that. I'm doing some Windows investigation ATM so this won't > be hard to add on to it. > > It's worth pointing out that the buildfarm client doesn't currently > test the syslogger at all. It probably should, at least optionally. > That wouldn't be too hard to arrange. A SMOP :-) > > Everything looks kosher on my Windows machine (tested both MSVC and Mingw64 builds) cheers andrew