]> 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:23 +0000 (17:34 -0400)
commitd8f15c95bec50f552ad0b13ea6eafb6975126184
tree95d00fbb1d15a1b40c0600ca7dd05e38f3ab3442
parent1e2bd43b31cdb6d0425430f1568814bce588d8fd
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