On Mon, 2010-04-19 at 10:24 -0400, Tom Lane wrote: > Robert Haas <robertmh...@gmail.com> writes: > > On Mon, Apr 19, 2010 at 5:05 AM, Simon Riggs <si...@2ndquadrant.com> wrote: > >> It doesn't seem to be something we should place highly on the list of > >> events we need protection from, does it? > > > Since when do we not protect against race-conditions just because > > they're low likelihood? > > Murphy's law says that the probability of any race condition happening > in the field is orders of magnitude higher than you think. This has > been proven true many times ...
Choices are 1. Check RecoveryInProgress() once outside of lock, plus wild rumour of Murphy 2. Check RecoveryInProgress() before and after holding lock 3. Check RecoveryInProgress() while holding lock All of which perform better than 4. Revert patch -- Simon Riggs www.2ndQuadrant.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
