On Fri, Nov 26, 2010 at 1:11 AM, Tom Lane <[email protected]> wrote: > Simon Riggs <[email protected]> writes: >> That would mean running GetCurrentTransactionId() inside LockAcquire() > >> if (lockmode >= AccessExclusiveLock && >> locktag->locktag_type == LOCKTAG_RELATION && >> !RecoveryInProgress()) >> (void) GetCurrentTransactionId(); > >> Any objections to that fix? > > Could we have a wal level test in there too please? It's pretty awful > in any case...
+1. Incidentally, I haven't been able to wrap my head around why we need to propagate AccessExclusiveLocks to the standby in the first place. Can someone explain? -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
