]> granicus.if.org Git - postgresql/commit
Make isolationtester more robust on locked commands
authorAlvaro Herrera <alvherre@alvh.no-ip.org>
Tue, 19 Jul 2011 18:22:42 +0000 (14:22 -0400)
committerAlvaro Herrera <alvherre@alvh.no-ip.org>
Tue, 19 Jul 2011 18:22:42 +0000 (14:22 -0400)
commitc8dfc892327b1a1e14efe110b0f1f267ef56c7a9
tree0197a9dc690cf28a042737d7d67ed76408299e19
parentd6db0e4e0e382cd5683a767424d05a0361af232a
Make isolationtester more robust on locked commands

Noah Misch diagnosed the buildfarm problems in the isolation tests
partly as failure to differentiate backends properly; the old code was
using backend IDs, which is not good enough because a new backend might
use an already used ID.  Use PIDs instead.

Also, the code was purposely careless about other concurrent activity,
because it isn't expected; and in fact, it doesn't affect the vast
majority of the time.  However, it can be observed that autovacuum can
block tables for long enough to cause sporadic failures.  The new code
accounts for that by ignoring locks held by processes not explicitly
declared in our spec file.

Author: Noah Misch
src/test/isolation/isolationtester.c