]> granicus.if.org Git - postgresql/commit
Reduce chatter from signaling of autovacuum workers.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 28 Jul 2015 21:34:00 +0000 (17:34 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 28 Jul 2015 21:34:00 +0000 (17:34 -0400)
commitf527c0a2a7c8507163bc22107c9d6a10b5f17174
treed59c070bfe7b709442431a9acde92b95e88161fe
parent444b2ebee97fdbabbb32ec7b4ebd1b6f40638ec5
Reduce chatter from signaling of autovacuum workers.

Don't print a WARNING if we get ESRCH from a kill() that's attempting
to cancel an autovacuum worker.  It's possible (and has been seen in the
buildfarm) that the worker is already gone by the time we are able to
execute the kill, in which case the failure is harmless.  About the only
plausible reason for reporting such cases would be to help debug corrupted
lock table contents, but this is hardly likely to be the most important
symptom if that happens.  Moreover issuing a WARNING might scare users
more than is warranted.

Also, since sending a signal to an autovacuum worker is now entirely a
routine thing, and the worker will log the query cancel on its end anyway,
reduce the message saying we're doing that from LOG to DEBUG1 level.

Very minor cosmetic cleanup as well.

Since the main practical reason for doing this is to avoid unnecessary
buildfarm failures, back-patch to all active branches.
src/backend/storage/lmgr/proc.c